refactor(parser): fetch 2 bytes in ? byte handler

[overlookmotel]

Lexer's ? byte handler needs to fetch next 2 bytes, so do that in one shot, rather than bounds-checking twice.

[graphite-app]

Your org has enabled the Graphite merge queue for merging into main

Add the label “merge” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

[overlookmotel]

• perf(parser): faster decoding unicode escapes in identifiers #4579
• perf(parser): fast path for ASCII when checking char after numeric literal #4577
• perf(parser): make not at EOF the hot path in Source methods #4576
• refactor(parser): convert lexer byte handler for | to a single match #4575
• refactor(parser): convert Lexer::read_minus to a single match #4574
• refactor(parser): convert Lexer::read_left_angle to a single match #4573
• perf(parser): optimize Lexer::hex_digit #4572
• perf(parser): speed up lexing non-decimal numbers #4571
• refactor(parser): add Lexer::consume_2_chars #4569
• refactor(parser): extract u8 not &u8 when iterating over bytes #4568
• refactor(parser): rename function #4566
• refactor(parser): rename vars #4565
• refactor(parser): rename function #4564
• refactor(parser): fetch 2 bytes in ? byte handler #4563 👈
• refactor(parser): shorten lexer code #4562
• perf(parser): use - not saturating_sub #4561
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @overlookmotel and the rest of your teammates on Graphite

[codspeed-hq]

CodSpeed Performance Report

Merging #4563 will not alter performance

_{Comparing 07-30-refactor_parser_fetch_2_bytes_in_byte_handler (0acc4a7) with main (7585e16)}

Summary

✅ 32 untouched benchmarks

[graphite-app]

Merge activity

• Jul 30, 8:24 PM EDT: Boshen added this pull request to the Graphite merge queue.
• Jul 30, 8:38 PM EDT: Boshen merged this pull request with the Graphite merge queue.