RegEx unable to parse "(\d)(?(1)yes|no)(\d)(\2)"

[pgovind]

Pattern: @"(\d)(?(1)yes|no)(\d)(\2)"
Input: "0yes11"
Current Match: "0yes"
Correct Match: "0yes11"

I found this while reading through our CountCaptures() routine. The bug is that we set _ignoreNextParens to true when we parse an alternation construct and never set it back to false. As a result, we don't detect any subsequent captures.

[pgovind]

Up for grabs if somebody wants it. Otherwise, I'll pick it up the next time I go through here.

[hannan-m]

Which version of the Dotnet core are you using? I just test tested with the same regex and input on .Net core 3.1 and it worked as expected.

[pgovind]

Interesting. I tested this on Regexstorm.net when I repro'd the bug. It seems to be working now. It could be that the new changes that went into the RegEx library fixed this. I'll review the code here and close this once I figure out why/how this was fixed.

[pgovind]

This actually works fine. _ignoreNextParens is being set to false in the next iteration, so we count the captures correctly. Closing this.