re_intuit_start(): don't unset MBOL on uselessness
authorDavid Mitchell <davem@iabyn.com>
Wed, 19 Mar 2014 17:37:20 +0000 (17:37 +0000)
committerDavid Mitchell <davem@iabyn.com>
Wed, 19 Mar 2014 18:25:55 +0000 (18:25 +0000)
commit6e88169dce11675b992a0c387588bd5a90a69a72
treec71df97a4f02f5d2d1e320fae846df5295f72f71
parent7d2d37f505b570402d76d76649cc2464812a5881
re_intuit_start(): don't unset MBOL on uselessness

When BmUSEFUL() for a pattern goes negative, we unset ~RXf_USE_INTUIT.

A bug fix in 2010 (c941595168) made this part of the code also remove
the RXf_ANCH_MBOL at the same time, if PREGf_IMPLICIT was set.

However, this was really just working round a bug in regexec_flags().
Once intuit was disabled, regexec_flags() would then start taking the
buggy code path.

This buggy code path was separately fixed in 2012 by 21eede782, so there's
no longer any need to remove this flag. So don't.
regexec.c