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

update regex-redux for c# #396

Merged
merged 4 commits into from
Jul 12, 2023
Merged

update regex-redux for c# #396

merged 4 commits into from
Jul 12, 2023

Conversation

danmoseley
Copy link
Contributor

general cleanup
change manual match counting to Regex.Count, which is more efficient.

I updated both the singleproc and multiproc ones although numbers seem to be for singleproc, which is much slower.

@danmoseley
Copy link
Contributor Author

cc @stephentoub fyi. hopefully this will give us updated numbers on https://programming-language-benchmarks.vercel.app/

@marelle65187512
Copy link

Your project is very popular among my team - we're considering using it for our next one.

@hanabi1224 hanabi1224 force-pushed the main branch 3 times, most recently from 3fc55d3 to d70acc7 Compare July 12, 2023 15:40
@hanabi1224 hanabi1224 enabled auto-merge (squash) July 12, 2023 16:14
@hanabi1224
Copy link
Owner

Thanks, sry for the delay and I might need more time to catch up with other PRs

@hanabi1224 hanabi1224 merged commit c47ccd8 into hanabi1224:main Jul 12, 2023
@danmoseley danmoseley deleted the redux1 branch July 12, 2023 16:37
@danmoseley
Copy link
Contributor Author

sure thing @hanabi1224 -- appreciate you running this -- it's more user friendly than the original CLBG. I like the idea of being able to submit PR's and the results automatically updated on the page.

my other PR enables more C# scenarios and switches to .NET 8.

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