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

A little more regex source generator tweaking #62605

Merged
merged 7 commits into from
Dec 10, 2021

Conversation

stephentoub
Copy link
Member

Couldn't help myself :) It's fun just looking at what code the generator writes for varying regexes and finding ways to improve it a bit here and there.

cc: @joperezr

@ghost
Copy link

ghost commented Dec 9, 2021

Tagging subscribers to this area: @dotnet/area-system-text-regularexpressions
See info in area-owners.md if you want to be subscribed.

Issue Details

Couldn't help myself :) It's fun just looking at what code the generator writes for varying regexes and finding ways to improve it a bit here and there.

cc: @joperezr

Author: stephentoub
Assignees: -
Labels:

area-System.Text.RegularExpressions

Milestone: -

@GSPP
Copy link

GSPP commented Dec 13, 2021

It's probably good to get quality high for the first release. After that, any further improvements could cause massive source control diffs when the generators are re-run.

@stephentoub
Copy link
Member Author

After that, any further improvements could cause massive source control diffs when the generators are re-run.

I'm not concerned about that. These source generators are intended to be run as part of every build, by default the source that's generated isn't even saved to disk, and even if you opt-in to saving it to disk, it's saved to a temporary location. Limiting innovation here because of concerns around improvements to the source generator would be akin to saying compilers shouldn't improve from version to version.

@ghost ghost locked as resolved and limited conversation to collaborators Jan 12, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants