From ab028a158ac4a4c6c0c2ebcca00ae595f93d2fe6 Mon Sep 17 00:00:00 2001 From: David Mitchell Date: Fri, 7 Feb 2014 17:04:21 +0000 Subject: [PATCH] re_intuit_start(): remove other_last = rx_origin The previous commit made this assignment conditional on other_last not decreasing; but it turns out that increasing it is pointless (although harmless), since the next time round the "other" substring block, the current rx_origin will be >= the old rx_origin (since we never decrease it), and s >= new rx_origin. So s would already be >= the value we would set other_last to, so it doesn't make any difference. --- regexec.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/regexec.c b/regexec.c index ed6f9fc..0634a8b 100644 --- a/regexec.c +++ b/regexec.c @@ -1121,8 +1121,6 @@ Perl_re_intuit_start(pTHX_ DEBUG_EXECUTE_r(PerlIO_printf(Perl_debug_log, " Found /%s^%s/m, restarting lookup for check-string at offset %ld...\n", PL_colors[0], PL_colors[1], (long)(t + 1 - strpos))); - if (other_last < rx_origin) - other_last = rx_origin; goto restart; } -- 2.7.4