pp_match(): remove redundant condition
authorDavid Mitchell <davem@iabyn.com>
Wed, 31 Jul 2013 09:08:18 +0000 (10:08 +0100)
committerDavid Mitchell <davem@iabyn.com>
Wed, 31 Jul 2013 09:08:18 +0000 (10:08 +0100)
a successful match always sets $-[0] now, so there's no need to check
whether its set

pp_hot.c

index 79c9c45..d7de38a 100644 (file)
--- a/pp_hot.c
+++ b/pp_hot.c
@@ -1447,14 +1447,11 @@ PP(pp_match)
         if (!mg) {
             mg = sv_magicext_mglob(TARG);
         }
-        assert(RX_OFFS(rx)[0].start != -1); /* XXX get rid of next line? */
-        if (RX_OFFS(rx)[0].start != -1) {
-            mg->mg_len = RX_OFFS(rx)[0].end;
-            if (RX_ZERO_LEN(rx))
-                mg->mg_flags |= MGf_MINMATCH;
-            else
-                mg->mg_flags &= ~MGf_MINMATCH;
-        }
+        mg->mg_len = RX_OFFS(rx)[0].end;
+        if (RX_ZERO_LEN(rx))
+            mg->mg_flags |= MGf_MINMATCH;
+        else
+            mg->mg_flags &= ~MGf_MINMATCH;
     }
 
     if ((!RX_NPARENS(rx) && !global) || gimme != G_ARRAY) {