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 is a major improvement for the pretty-printer.
The main change is that
TokenStreamTransformer.prettyprint
now takes a Prism token stream instead of a simplified token stream. This means that it has a lot more information to work with and it uses that information.It mostly behaves like the old pretty-printer with 3 main differences:
Example:
["namespace", ["java"]]
Example:
["number", "100"], ["operator", ".."], ["number", "100"]
If anyone wants to know about how it works in detail, I'll explain, but it's a bit much so I won't do so here.
I also want to ask whether I should reformat all
.test
files.The pretty-printer is quite good and it would be nice if all test files were formatted consistently. However, 50% of all test files change when reformatted.