pp_subst: don't use REXEC_COPY_STR on 2nd match
authorDavid Mitchell <davem@iabyn.com>
Mon, 15 Jul 2013 20:57:34 +0000 (21:57 +0100)
committerDavid Mitchell <davem@iabyn.com>
Sun, 28 Jul 2013 09:33:39 +0000 (10:33 +0100)
commitfeb38e3b9dba8f9f75fe6c737d7c4d99ff1aca46
tree1c633eb07e2c3bdf394ef1e0ad4c082c4a117a77
parent2ec7214c34f5d160ff6a0e8acb3f151c5974e83a
pp_subst: don't use REXEC_COPY_STR on 2nd match

pp_subst() sets the REXEC_COPY_STR flag on the first match. On the second
and subsequent matches, it doesn't set it in two out three of the branches
(including pp_susbstcont) where it calls CALLREGEXEC().
The one place where it *does* set it is a (harmless) mistake, since regexec
ignores REXEC_COPY_STR if REXEC_NOT_FIRST is set (which is it is, on all 3
brnanches).

So unset REXEC_COPY_STR in the third branch too, for consistency
pp_hot.c