Skip to content

Conversation

@bd82
Copy link
Member

@bd82 bd82 commented Dec 20, 2025

TODO:

  • Code Review
  • Verify if any more docs need to be updated regarding lexer optimizations / performance (reduce vs full disabling of optimizations)
  • Should not test case ["lookbehind regexp", /(?<!a)d/] fail now that regexp-to-ast supports lookbehind syntax?
    - Because lookbehind support was not merged (Adds lookbehind support (#2049) #2134)
    - test case fails as expected and was removed as it is no longer relevant.
  • Verify no performance regressions
    • There may be a small regression in the more complex CSS lexer
    • image
    • But it may be worth "paying" for avoiding a major regression anytime there is an UN-optimizeable token pattern.

@bd82 bd82 force-pushed the lexer_unoptimized branch from 3906883 to 53bd2a5 Compare December 20, 2025 14:35
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