Skip to content

kacperek18 - #635

Open
Kacperek18 wants to merge 97 commits into
620-incorrectly-tagged-default-implementation-keyword-outside-of-type-definitionfrom
master
Open

Kacperek18 wants to merge 97 commits into
620-incorrectly-tagged-default-implementation-keyword-outside-of-type-definitionfrom
master

Conversation

@Kacperek18

Copy link
Copy Markdown

replaces the commands/ + settings/ fornix setup with a xome flake and
run/ scripts, matching better-perl-syntax and better-nix-syntax.

two deviations from those repos:
- python3 instead of python2, because python2 fails to build on
  aarch64-darwin (gnu patch OOMs from the builder's unlimited nofile)
- node-gyp 10 as a devDependency, since textmate-tester hoists node-gyp 4
  which refuses python3 when compiling oniguruma
- char8_t/char16_t/char32_t are now primitive types instead of falling
  through to the generic *_t posix-reserved rule (issue 678)
- doxygen command is \copydetails, not \copydetail (issue 624)
- printf %b/%B binary conversion from C23 (issue 700)
- alias declarations in a for-init statement, ie `for (using a = int;;)`
  from C++23 (issue 689)
- issue 631: `enum:int` and `enum class Color:unsigned` failed to
  highlight the enum-key when no space preceded the colon
- issue 692: `~modem_t(void)` left `void` unhighlighted, since the
  destructor parameter range had no includes at all
The shared inline_scope_resolution pattern used a possessive quantifier,
so in `Foo::Foo()` / `Foo::~Foo()` it swallowed the `Foo::` that the
following backreference needed, and the match could never complete.
These definitions fell through to a generic function call, which also
left their parameters untagged (issue 509).

Scope resolution is now non-possessive for the constructor/destructor
patterns only; everything else keeps the possessive form. Timing on the
full example suite is unchanged.

Recognizing these constructors exposed that the member initializer list
did not allow preprocessor directives, so `#if`/`#endif` between
initializers lost their highlighting (issue 528).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants