The position of the modifier flag bits is actually encoded by a right
authorNicholas Clark <nick@ccl4.org>
Sat, 29 Dec 2007 15:11:04 +0000 (15:11 +0000)
committerNicholas Clark <nick@ccl4.org>
Sat, 29 Dec 2007 15:11:04 +0000 (15:11 +0000)
shift 12 in two places, so replace that magic number with a macro
RXf_PMf_STD_PMMOD_SHIFT defined adjacent to the flags it interacts
with.

p4raw-id: //depot/perl@32774

ext/re/re.xs
regcomp.c
regexp.h

index 94cb2f5..72e4a70 100644 (file)
@@ -107,7 +107,8 @@ PPCODE:
 
             const char *fptr = INT_PAT_MODS;
             char ch;
-            U16 match_flags = (U16)((re->extflags & PMf_COMPILETIME) >> 12);
+            U16 match_flags = (U16)((re->extflags & PMf_COMPILETIME)
+                                   >> RXf_PMf_STD_PMMOD_SHIFT);
 
             while((ch = *fptr++)) {
                 if(match_flags & 1) {
index 070b183..10e63f5 100644 (file)
--- a/regcomp.c
+++ b/regcomp.c
@@ -4286,7 +4286,8 @@ redo_first_pass:
         bool has_p     = ((r->extflags & RXf_PMf_KEEPCOPY) == RXf_PMf_KEEPCOPY);
        bool has_minus = ((r->extflags & RXf_PMf_STD_PMMOD) != RXf_PMf_STD_PMMOD);
        bool has_runon = ((RExC_seen & REG_SEEN_RUN_ON_COMMENT)==REG_SEEN_RUN_ON_COMMENT);
-       U16 reganch = (U16)((r->extflags & RXf_PMf_STD_PMMOD) >> 12);
+       U16 reganch = (U16)((r->extflags & RXf_PMf_STD_PMMOD)
+                           >> RXf_PMf_STD_PMMOD_SHIFT);
        const char *fptr = STD_PAT_MODS;        /*"msix"*/
        char *p;
         RX_WRAPLEN(r) = plen + has_minus + has_p + has_runon
index 60606a5..2958875 100644 (file)
--- a/regexp.h
+++ b/regexp.h
@@ -255,6 +255,7 @@ and check for NULL.
 #define RXf_PMf_EXTENDED       0x00008000 /* /x         */
 #define RXf_PMf_KEEPCOPY       0x00010000 /* /p         */
 /* these flags are transfered from the PMOP->op_pmflags member during compilation */
+#define RXf_PMf_STD_PMMOD_SHIFT        12
 #define RXf_PMf_STD_PMMOD      (RXf_PMf_MULTILINE|RXf_PMf_SINGLELINE|RXf_PMf_FOLD|RXf_PMf_EXTENDED)
 #define RXf_PMf_COMPILETIME    (RXf_PMf_MULTILINE|RXf_PMf_SINGLELINE|RXf_PMf_LOCALE|RXf_PMf_FOLD|RXf_PMf_EXTENDED|RXf_PMf_KEEPCOPY)