-
Notifications
You must be signed in to change notification settings - Fork 888
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rustfmt breaks trait method when where clause is commented out #4646
Labels
Milestone
Comments
vallentin
added a commit
to vallentin/rustfmt
that referenced
this issue
Jan 17, 2021
vallentin
added a commit
to vallentin/rustfmt
that referenced
this issue
Jan 17, 2021
vallentin
added a commit
to vallentin/rustfmt
that referenced
this issue
Jan 17, 2021
calebcartwright
pushed a commit
that referenced
this issue
Jan 18, 2021
calebcartwright
added
the
1x-backport:pending
Fixed/resolved in source but not yet backported to a 1x branch and release
label
Jan 18, 2021
davidBar-On
pushed a commit
to davidBar-On/rustfmt
that referenced
this issue
Jan 18, 2021
calebcartwright
added
1x-backport:completed
and removed
1x-backport:pending
Fixed/resolved in source but not yet backported to a 1x branch and release
labels
Jan 28, 2021
calebcartwright
pushed a commit
to calebcartwright/rustfmt
that referenced
this issue
Jan 28, 2021
calebcartwright
pushed a commit
that referenced
this issue
Jan 28, 2021
JohnTitor
added a commit
to JohnTitor/rust
that referenced
this issue
Jan 30, 2021
…anxiyn update rustfmt to v1.4.34 Short summary: Various formatting fixes (several const generic related) and introduction of `imports_granularity` config option Long summary copied from changelog: #### Changed - `merge_imports` configuration has been deprecated in favor of the new `imports_granularity` option. Any existing usage of `merge_imports` will be automatically mapped to the corresponding value on `imports_granularity` with a warning message printed to encourage users to update their config files. #### Added - New `imports_granularity` option has been added which succeeds `merge_imports`. This new option supports several additional variants which allow users to merge imports at different levels (crate or module), and even flatten imports to have a single use statement per item. ([PR rust-lang/rustfmt#4634](rust-lang/rustfmt#4634), [PR rust-lang/rustfmt#4639](rust-lang/rustfmt#4639)) See the section on the configuration site for more information https://rust-lang.github.io/rustfmt/?version=v1.4.33&search=#imports_granularity #### Fixed - Fix erroneous removal of `const` keyword on const trait impl ([rust-lang/rustfmt#4084](rust-lang/rustfmt#4084)) - Fix incorrect span usage wit const generics in supertraits ([rust-lang/rustfmt#4204](rust-lang/rustfmt#4204)) - Use correct span for const generic params ([rust-lang/rustfmt#4263](rust-lang/rustfmt#4263)) - Correct span on const generics to include type bounds ([rust-lang/rustfmt#4310](rust-lang/rustfmt#4310)) - Idempotence issue on blocks containing only empty statements ([rust-lang/rustfmt#4627](rust-lang/rustfmt#4627) and [rust-lang#3868](rust-lang/rustfmt#3868)) - Fix issue with semicolon placement on required functions that have a trailing comment that ends in a line-style comment before the semicolon ([rust-lang/rustfmt#4646](rust-lang/rustfmt#4646)) - Avoid shared interned cfg_if symbol since rustfmt can re-initialize the rustc_ast globals on multiple inputs ([rust-lang/rustfmt#4656](rust-lang/rustfmt#4656)) - Don't insert trailing comma on (base-less) rest in struct literals within macros ([rust-lang/rustfmt#4675](rust-lang/rustfmt#4675))
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
In short, if you comment out the where clause, then the subsequent semicolon is moved into the comment.
Which of course breaks the code.
Before:
After:
rustfmt 1.4.25-stable (0f29ff6 2020-11-11)
The text was updated successfully, but these errors were encountered: