-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
Document SRM regex approach/algorithms/terminology in README.md #60917
Comments
Tagging subscribers to this area: @eerhardt, @dotnet/area-system-text-regularexpressions Issue Detailsnull
|
That README file has now been deleted as it was very out-of-date already given all the changes that went in for 7.0. That said, there are several issues in Regex that are documentation only that are meant to call out these kinds of designs. Do you think we should add an updated README back? or should we document this in the code via comments or perhaps via remarks on the relevant types? |
Comments in code are fine (preferable, IMHO) |
I'm going to close this. There are a lot of comments already, and while more would be wonderful and encouraged, it doesn't need to be tracked by an issue anymore nor block 7.0's release. |
System.Text.RegularExpressions has a README.md file that provides an overview of the library's design. That should be updated, including adding details on the new NonBacktracking implementation and key terminology/algorithms used throughout it.
The text was updated successfully, but these errors were encountered: