Matthew Fosdick

Looks good, couple changes. 1. use `.then(std_space)` rather than `.maybe(@spaces)`. This is for a couple reasons: - `std_space` supports inline comments - `std_space` is automatically optional - `.maybe(@spaces)` can catastrophically...

If a keyword doesn't influence any other highlighting, then all that is need is to add the correct flags inside tokens.rb. In the new PR, I added support for import.

Yes `bundle install`, I should probably add a post install hook.

It is expected that `requires` is not usable as a function name due to, `avoid_invalid_function_names = @cpp_tokens.lookBehindToAvoidWordsThat(:isWord, not(:isPreprocessorDirective), not(:isValidFunctionName))`. Can you share what you have for the requires highlighting? As...

If you are using the debugger could you check that you are using the "Launch and Build All" configuration, the default "Launch Extension" does not usually rebuild the syntax files.

That is very strange, If I add the requires token directly after `consteval` (line 221) everything seems to work.  Could you remove the...

There shouldn't be, `npm test` has no knowledge of what vscode loads normally, and the extension development host replaces the extension, so that the default one should never be loaded.

Is the pattern being recognized as a lambda? (Is the scope at `{` `meta.function.definition.body.lambda.cpp` when using `Developer: Inspect TM Scopes`

Could you try to reproduce this from a clean clone of the repo to try to determine if this is a build issue or if other changes might be affecting...

After testing on wxWidgets, llvm, qt, and boost, I found very little false positives, and those were eliminated when, I limited the include guard detection to the first detected. Edit:...