Skip to content

Regular Expressions Improvements #135

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

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

Regular Expressions Improvements #135

wants to merge 10 commits into from

Conversation

Mzack9999
Copy link
Member

@Mzack9999 Mzack9999 commented May 14, 2025

RE2 brings x2 speed but more allocs. Some pattern cannot be compiled so we fallback to standard library

Closes #112

@Mzack9999 Mzack9999 self-assigned this May 14, 2025
@ehsandeep ehsandeep requested a review from dogancanbakir May 14, 2025 09:32
@Mzack9999 Mzack9999 marked this pull request as draft May 14, 2025 09:34
@ehsandeep ehsandeep removed the request for review from dogancanbakir May 14, 2025 09:36
@Mzack9999 Mzack9999 marked this pull request as ready for review May 14, 2025 15:14
@Mzack9999 Mzack9999 requested a review from dogancanbakir May 14, 2025 15:15
@dogancanbakir dogancanbakir requested a review from ehsandeep May 15, 2025 13:52
@ehsandeep ehsandeep requested a review from Ice3man543 June 20, 2025 13:03
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.

go-regexp is slow causing high cpu usage and slowness
2 participants