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
Use Regex source generator #2794
Comments
martincostello
added
the
help-wanted
A change up for grabs for contributions from the community
label
Apr 14, 2024
martincostello
removed
the
help-wanted
A change up for grabs for contributions from the community
label
Apr 14, 2024
Turns out those are the only static occurences in the whole code base. All the others are based on dynamic inputs. |
martincostello
added a commit
to martincostello/Swashbuckle.AspNetCore
that referenced
this issue
Apr 14, 2024
Use the Regex source generator for .NET 7+. Resolves domaindrivendev#2794.
Merged
martincostello
added a commit
to martincostello/Swashbuckle.AspNetCore
that referenced
this issue
Apr 16, 2024
Use the Regex source generator for .NET 7+. Resolves domaindrivendev#2794.
martincostello
added a commit
to martincostello/Swashbuckle.AspNetCore
that referenced
this issue
Apr 23, 2024
Use the Regex source generator for .NET 7+. Resolves domaindrivendev#2794.
martincostello
added a commit
to martincostello/Swashbuckle.AspNetCore
that referenced
this issue
May 1, 2024
Use the Regex source generator for .NET 7+. Resolves domaindrivendev#2794.
martincostello
added a commit
to martincostello/Swashbuckle.AspNetCore
that referenced
this issue
May 8, 2024
Use the Regex source generator for .NET 7+. Resolves domaindrivendev#2794.
How much perf boost after your changes? |
I didn't measure. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
For
net7.0
and later we can look to leverage the Regex source generator to improve performance.For example:
Swashbuckle.AspNetCore/src/Swashbuckle.AspNetCore.SwaggerGen/XmlComments/XmlCommentsTextHelper.cs
Lines 10 to 13 in 830c5a6
The text was updated successfully, but these errors were encountered: