Regex: reduce allocation slightly, add tests, code cleanup, add parser comments ...
authorViktor Hofer <viktor.hofer@microsoft.com>
Fri, 29 Jun 2018 15:47:53 +0000 (17:47 +0200)
committerAnirudh Agnihotry <anirudhagnihotry098@gmail.com>
Sat, 30 Jun 2018 02:07:41 +0000 (19:07 -0700)
commitdbaf2957387c5290a680c8918779683194137b1d
tree5e69e6b82da3968da09a6d4cff474caff8622226
parentc7bfd35ab09306885ae74c760ac69ec42be42557
Regex: reduce allocation slightly, add tests, code cleanup, add parser comments (#30632)

* RegexParser & optionsstack ref

* Add test coverage for Group.Synchronized

* Adjust options mode test case

* Add inline comment '#' test branch

* Add comments

* Replace manual ToLower calls by Span.ToLower

* Make applicable fields readonly in parser

* Change to Assert to reduce an if check in one branch

* Code formatting

* Avoid string allocation when IgnoreCase set

Prefix patterns which are passed to RegexBoyerMoore are already
lowercased by the parser. Remove the redundant ToLower() call and assert
the patterns lowercase state

* Add surrogate pair positive & negative tests

* Add test cases for rtl anchor

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
src/System.Private.CoreLib/shared/System/Collections/Generic/ValueListBuilder.cs