Fix #1237 cpp whitespace token usage expanded#1904
Conversation
|
I did not run the tests initially... The proposed change impacts more places than i thought initially. It seems there are also more cases of Keyword + Text instead of Keyword + Text.Whitespace. (run grep -r "Keyword, Text" in pygments/lexers) |
|
I've been trying to slowly clean up existing lexers and be more attentive to |
This sounds like a problem i can handle. Just a question for the process, would you like individual PRs per changed lexers? |
|
Merged, thanks a lot for working on this! |
|
Thanks for picking this up! I think one PR for everything should be fine given it's fairly easy to review. If you're building any scripts to automate it, it would be great if you could submit them as well should we ever need to go through this exercise again. |
It seems likely, that the original issuer had a highlighting theme where the token
Texthas been highlighted.Whereever applicable, the
Texttoken is replaced with the more specificWhitespacetoken - not just theclasskeyword, but also similar occurrences, such as enums and structs.This should fix #1237, and should generally be more precise lexing as well.