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.

Sign up for GitHub

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

Jump to bottom

Regex parser nits #88558

Merged
merged 20 commits into from
Jul 14, 2023
Merged

Regex parser nits #88558

Changes from all commits
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
378c0b0
inline stack
danmoseley Jul 9, 2023
81a1353
Inline CaptureSlotFromName()
danmoseley Jul 9, 2023
014ea9a
Inline HexDigit(c)
danmoseley Jul 9, 2023
d5608a4
lint
danmoseley Jul 9, 2023
af5ded3
inline OptionFromCode
danmoseley Jul 9, 2023
03ae945
inline IsCaptureName()
danmoseley Jul 9, 2023
0268658
build break
danmoseley Jul 10, 2023
1a13b44
IndexOf in ScanReplacement()
danmoseley Jul 10, 2023
9f63918
Remove unnecessary clause
danmoseley Jul 10, 2023
1c1c49f
Inline AddConcatenate(bool lazy, int min, int max)
danmoseley Jul 10, 2023
c65534f
Change to IndexOf in 2 places
danmoseley Jul 10, 2023
53a1fbe
Use StartsWith in 1 place
danmoseley Jul 10, 2023
7883ae2
remove low value assert
danmoseley Jul 10, 2023
fbdad9b
low value asserts
danmoseley Jul 10, 2023
5251fa4
minor simplify
danmoseley Jul 10, 2023
5404f7f
Inline AddUnitToConcatenate()
danmoseley Jul 10, 2023
70bdb4f
revert starstwith
danmoseley Jul 10, 2023
550b255
Update RegexParser.cs
danmoseley Jul 10, 2023
7271a68
Update RegexParser.cs
danmoseley Jul 10, 2023
fb85f73
Update RegexParser.cs
danmoseley Jul 14, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Loading