-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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
Fix repr pretty display #70075
Fix repr pretty display #70075
Conversation
We need to take two tokens into consideration if we want to pretty-print token streams in a nicer way. 1 + (2 + 3) |
I can check if the previous one is an ident (which would make sense). What do you think? |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
e341d31
to
313ec4f
Compare
I updated it as suggested: I don't prepend a whitespace in case the previous token is an ident. |
r=me with squashed commits and green CI. |
313ec4f
to
9a017da
Compare
@bors: r=petrochenkov |
📌 Commit 9a017da has been approved by |
…petrochenkov Fix repr pretty display Fixes rust-lang#70027. r? @varkor
…petrochenkov Fix repr pretty display Fixes rust-lang#70027. r? @varkor
Rollup of 10 pull requests Successful merges: - rust-lang#67749 (keyword docs for else and inkeyword docs for else and in.) - rust-lang#69139 (clean up E0308 explanation) - rust-lang#69189 (Erase regions in writeback) - rust-lang#69837 (Use smaller discriminants for generators) - rust-lang#69838 (Expansion-driven outline module parsing) - rust-lang#69839 (Miri error reform) - rust-lang#69899 (Make methods declared by `newtype_index` macro `const`) - rust-lang#69920 (Remove some imports to the rustc crate) - rust-lang#70075 (Fix repr pretty display) - rust-lang#70106 (Tidy: fix running rustfmt twice) Failed merges: - rust-lang#70051 (Allow `hir().find` to return `None`) - rust-lang#70074 (Expand: nix all fatal errors) r? @ghost
Fixes #70027.
r? @varkor