Skip to content

refactor(parser): macro for ASCII identifier byte handlers#2351

Merged
Boshen merged 1 commit intomainfrom
02-08-refactor_parser_macro_for_ASCII_identifier_byte_handlers
Feb 9, 2024
Merged

refactor(parser): macro for ASCII identifier byte handlers#2351
Boshen merged 1 commit intomainfrom
02-08-refactor_parser_macro_for_ASCII_identifier_byte_handlers

Conversation

@overlookmotel
Copy link
Copy Markdown
Member

@overlookmotel overlookmotel commented Feb 8, 2024

Add a macro for ASCII identifier byte handlers.

This is a preparatory step towards #2352.

@overlookmotel
Copy link
Copy Markdown
Member Author

overlookmotel commented Feb 8, 2024

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Feb 8, 2024

CodSpeed Performance Report

Merging #2351 will not alter performance

Comparing 02-08-refactor_parser_macro_for_ASCII_identifier_byte_handlers (beafc6c) with main (6f597b1)

Summary

✅ 27 untouched benchmarks

Base automatically changed from 02-08-refactor_parser_all_pointer_manipulation_through_SourcePosition_ to main February 9, 2024 02:26
Copy link
Copy Markdown
Member

Boshen commented Feb 9, 2024

Merge activity

  • Feb 8, 10:50 PM EST: @Boshen started a stack merge that includes this pull request via Graphite.
  • Feb 8, 10:51 PM EST: Graphite rebased this pull request as part of a merge.
  • Feb 8, 10:55 PM EST: @Boshen merged this pull request with Graphite.

@Boshen Boshen force-pushed the 02-08-refactor_parser_macro_for_ASCII_identifier_byte_handlers branch from 593a602 to beafc6c Compare February 9, 2024 03:51
@Boshen Boshen merged commit 6910e4f into main Feb 9, 2024
@Boshen Boshen deleted the 02-08-refactor_parser_macro_for_ASCII_identifier_byte_handlers branch February 9, 2024 03:55
IWANABETHATGUY pushed a commit to IWANABETHATGUY/oxc that referenced this pull request May 29, 2024
…ct#2351)

Add a macro for ASCII identifier byte handlers.

This is a preparatory step towards oxc-project#2352.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-parser Area - Parser

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants