PATCH: [perl #120041] regcomp.c missing parens and broken STCLASS
authorKarl Williamson <public@khwilliamson.com>
Mon, 30 Sep 2013 20:06:23 +0000 (14:06 -0600)
committerKarl Williamson <public@khwilliamson.com>
Mon, 30 Sep 2013 20:26:41 +0000 (14:26 -0600)
commit77ebeeba5f5d47be96b3cef485cb98b1c4980510
treee0bdcbcdf978645f1e14da19a55687c07b114f1a
parent79d51b8d94a1a8bcf282ea7b652906d0a4233bfc
PATCH: [perl #120041] regcomp.c missing parens and broken STCLASS

This was caused by a problem in commit
43a64b8b430604bd4d76fd256a5babdccaf0ab2b
which was masked by problems in commit
a0dd42312a1f26356d2fdf49656e45b77c2cefb5.

The first commit deleted a line it shouldn't have.  I thought that a
variable wasn't modified before reaching that line, but it turns out it
was because another variable is set to point to its address, and was
modified inside a subroutine call.  The other two lines deleted in the
first commit are ok to have deleted.

The second commit masked the problems of the first by omitting some
necessary grouping parentheses, which caused things to not work as
intended, and we didn't have a good test case in our suite to find this
problem.  (Thanks to Lukas Mai for spotting the problem and submitting a
test case.)
regcomp.c
t/re/re_tests