Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor(lexer): introduce native patterns for custom lexing logic #1025

Merged
merged 1 commit into from
Dec 2, 2024

Conversation

gvozdvmozgu
Copy link
Collaborator

No description provided.

This comment was marked as outdated.

@gvozdvmozgu gvozdvmozgu force-pushed the refactor/replace-fancy-regex-with-native branch from e9c7f8c to b7d4f4a Compare December 2, 2024 15:09

This comment was marked as outdated.

@gvozdvmozgu gvozdvmozgu force-pushed the refactor/replace-fancy-regex-with-native branch from b7d4f4a to 28ba915 Compare December 2, 2024 15:29
@gvozdvmozgu gvozdvmozgu changed the title Refactor/replace fancy regex with native refactor(lexer): introduce native patterns for custom lexing logic Dec 2, 2024
Copy link

github-actions bot commented Dec 2, 2024

Benchmark for a592ae0

Click to view benchmark
Test Base PR %
DepthMap::from_parent 41.0±0.26µs 40.8±0.34µs -0.49%
fix_complex_query 10.4±0.22ms 10.6±0.18ms +1.92%
parse_complex_query 2.8±0.03µs 2.8±0.04µs 0.00%
parse_expression_recursion 5.1±0.06µs 5.0±0.07µs -1.96%
parse_simple_query 804.0±7.84ns 836.9±18.86ns +4.09%

@gvozdvmozgu gvozdvmozgu marked this pull request as ready for review December 2, 2024 16:09
@gvozdvmozgu gvozdvmozgu requested a review from benfdking December 2, 2024 16:09
@benfdking benfdking merged commit c2a1ea3 into main Dec 2, 2024
36 checks passed
@benfdking benfdking deleted the refactor/replace-fancy-regex-with-native branch December 2, 2024 16:38
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.

2 participants