restyle to match dropshot, omicron, plus general style clean-up #35
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This change does a one-time application of the following rustfmt changes:
This is just to switch to C-style comments and rewrap strings and comments at 80 columns. I did these in separate commits so we could evaluate the options separately in case we don't like them. @andrewjstone what do you think?
We don't want to actually update rustfmt.toml for all the reasons discussed in oxidecomputer/dropshot#77.
If anybody has an outstanding change that they want to sync with this, I believe the thing to do is: immediately before sync'ing up with this change, commit any local changes, copy the above config lines (plus
unstable_features = true
) into your rustfmt.toml by hand, runcargo +nightly fmt
to restyle your clone, remove the local changes to rustfmt.toml, commit the restyled version, and then sync up. I believe if you do this you will avoid conflicts from this change.