regcomp.c: Don't set /i in start class unless /l
authorKarl Williamson <public@khwilliamson.com>
Sun, 14 Oct 2012 00:17:11 +0000 (18:17 -0600)
committerKarl Williamson <public@khwilliamson.com>
Sun, 14 Oct 2012 15:03:38 +0000 (09:03 -0600)
commitae6bdd37b100ecaba3f858c0375997681062d5b6
tree8b14edb29ee283fa001eda8ba2a7b1481befa111
parent969c44e74a39672c48f079853d5eba9f1870e860
regcomp.c: Don't set /i in start class unless /l

There is a deficiency in the optimizer in which it doesn't get rid of
flags that it should.  One of these is if it should match /i or not.
Currently it always (perhaps not quite, I don't know) assumes that it
should match under /i, yielding false positives and slowing things down.
But a recent commit changed the flag that tells it to do this, so that it
only gets set if /l is also specified.  There is already existing code to
work around the optimizer deficiency for /l.  This commit just moves the
/i flag handling to that existing code, so it won't get invoked unless
/l is specified.
regcomp.c