2007-11-24 Eric Seidel <eric@webkit.org>
authoreric@webkit.org <eric@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 29 Nov 2007 11:09:42 +0000 (11:09 +0000)
committereric@webkit.org <eric@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 29 Nov 2007 11:09:42 +0000 (11:09 +0000)
        Reviewed by Sam.

        removing more macros

        * pcre/pcre_compile.cpp:
        (could_be_empty_branch):
        (compile_branch):
        (calculateCompiledPatternLengthAndFlags):
        * pcre/pcre_exec.cpp:
        (match):
        (jsRegExpExecute):
        * pcre/pcre_internal.h:
        * pcre/pcre_xclass.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@28144 268f45cc-cd09-0410-ab3c-d52691b4dbfc

JavaScriptCore/ChangeLog
JavaScriptCore/pcre/pcre_compile.cpp
JavaScriptCore/pcre/pcre_exec.cpp
JavaScriptCore/pcre/pcre_internal.h
JavaScriptCore/pcre/pcre_xclass.cpp

index edb9be59a73ae53a6691d346e09a82f0cfbde5ba..a10e7b937f889eb1477a1a5805f4efa5eb8cedb8 100644 (file)
@@ -1,3 +1,19 @@
+2007-11-24  Eric Seidel  <eric@webkit.org>
+
+        Reviewed by Sam.
+
+        removing more macros
+
+        * pcre/pcre_compile.cpp:
+        (could_be_empty_branch):
+        (compile_branch):
+        (calculateCompiledPatternLengthAndFlags):
+        * pcre/pcre_exec.cpp:
+        (match):
+        (jsRegExpExecute):
+        * pcre/pcre_internal.h:
+        * pcre/pcre_xclass.cpp:
+
 2007-11-24  Eric Seidel  <eric@webkit.org>
 
         Reviewed by Maciej.
index c615669f1f9adf26af194d10f889e995811e311c..92d272ae84d6a77d70bf8b12caea4e48b5c8f0c8 100644 (file)
@@ -130,7 +130,7 @@ static const char* error_text(ErrorCode code)
 
 /* Definition to allow mutual recursion */
 
-static BOOL
+static bool
   compile_regex(int, int *, uschar **, const UChar**, const UChar*, ErrorCode*, int,
     int *, int *, compile_data *);
 
@@ -158,7 +158,7 @@ Returns:         zero or positive => a data character
 
 static int
 check_escape(const UChar** ptrptr, const UChar* patternEnd, ErrorCode* errorcodeptr, int bracount,
-  BOOL isclass)
+  bool isclass)
 {
 const UChar* ptr = *ptrptr + 1;
 int i;
@@ -304,7 +304,7 @@ Arguments:
 Returns:    true or false
 */
 
-static BOOL
+static bool
 is_counted_repeat(const UChar* p, const UChar* patternEnd)
 {
 if (p >= patternEnd || !isASCIIDigit(*p))
@@ -413,7 +413,7 @@ Returns:       pointer to the first significant opcode
 */
 
 static const uschar*
-first_significant_code(const uschar *code, BOOL skipassert)
+first_significant_code(const uschar *code, bool skipassert)
 {
 for (;;)
   {
@@ -623,7 +623,7 @@ Arguments:
 Returns:      true if what is matched could be empty
 */
 
-static BOOL
+static bool
 could_be_empty_branch(const uschar *code, const uschar *endcode)
 {
 int c;
@@ -637,7 +637,7 @@ for (code = first_significant_code(code + 1 + LINK_SIZE, true);
 
   if (c >= OP_BRA)
     {
-    BOOL empty_branch;
+    bool empty_branch;
     if (GET(code, 1) == 0) return true;    /* Hit unclosed bracket */
 
     /* Scan a closed bracket */
@@ -833,7 +833,7 @@ Returns:         true on success
                  false, with *errorcodeptr set non-zero on error
 */
 
-static BOOL
+static bool
 compile_branch(int options, int *brackets, uschar **codeptr,
                const UChar** ptrptr, const UChar* patternEnd, ErrorCode* errorcodeptr, int *firstbyteptr,
                int *reqbyteptr, compile_data *cd)
@@ -846,14 +846,14 @@ compile_branch(int options, int *brackets, uschar **codeptr,
     int c;
     uschar *code = *codeptr;
     uschar *tempcode;
-    BOOL groupsetfirstbyte = false;
+    bool groupsetfirstbyte = false;
     const UChar* ptr = *ptrptr;
     const UChar* tempptr;
     uschar *previous = NULL;
     uschar *previous_callout = NULL;
     uschar classbits[32];
     
-    BOOL class_utf8;
+    bool class_utf8;
     uschar *class_utf8data;
     uschar utf8_char[6];
     
@@ -883,9 +883,9 @@ compile_branch(int options, int *brackets, uschar **codeptr,
     
     for (;; ptr++)
     {
-        BOOL negate_class;
-        BOOL should_flip_negation; /* If a negative special such as \S is used, we should negate the whole class to properly support Unicode. */
-        BOOL is_quantifier;
+        bool negate_class;
+        bool should_flip_negation; /* If a negative special such as \S is used, we should negate the whole class to properly support Unicode. */
+        bool is_quantifier;
         int class_charcount;
         int class_lastchar;
         int skipbytes;
@@ -2021,7 +2021,7 @@ Argument:
 Returns:      true on success
 */
 
-static BOOL
+static bool
 compile_regex(int options, int* brackets, uschar** codeptr,
               const UChar** ptrptr, const UChar* patternEnd, ErrorCode* errorcodeptr, int skipbytes,
               int* firstbyteptr, int* reqbyteptr, compile_data* cd)
@@ -2175,7 +2175,7 @@ Arguments:
 Returns:     true or false
 */
 
-static BOOL
+static bool
 is_anchored(const uschar *code, int options, unsigned int bracket_map,
   unsigned int backref_map)
 {
@@ -2235,7 +2235,7 @@ Arguments:
 Returns:         true or false
 */
 
-static BOOL
+static bool
 is_startline(const uschar *code, unsigned int bracket_map,
   unsigned int backref_map)
 {
@@ -2302,7 +2302,7 @@ Returns:     -1 or the fixed first char
 */
 
 static int
-find_firstassertedchar(const uschar *code, int options, BOOL inassert)
+find_firstassertedchar(const uschar *code, int options, bool inassert)
 {
 int c = -1;
 do {
@@ -2364,8 +2364,8 @@ static int calculateCompiledPatternLengthAndFlags(const UChar* pattern, int patt
     int branch_extra = 0;
     int branch_newextra;
     int lastitemlength = 0;
-    BOOL class_utf8;
-    BOOL capturing;
+    bool class_utf8;
+    bool capturing;
     unsigned int brastackptr = 0;
     int brastack[BRASTACK_SIZE];
     uschar bralenstack[BRASTACK_SIZE];
index 1748a77eec77d366fef44fb989cb108659179a4b..c66261bf4580bed6c44dfbcb58dd6601e07ee441 100644 (file)
@@ -124,13 +124,13 @@ typedef struct match_data {
   int    offset_max;            /* The maximum usable for return data */
   const uschar *lcc;            /* Points to lower casing table */
   const uschar *ctypes;         /* Points to table of type maps */
-  BOOL   offset_overflow;       /* Set if too many extractions */
+  bool   offset_overflow;       /* Set if too many extractions */
   UChar*  start_subject;         /* Start of the subject string */
   UChar*  end_subject;           /* End of the subject string */
   UChar*  end_match_ptr;         /* Subject position at end match */
   int    end_offset_top;        /* Highwater mark at end of match */
-  BOOL   multiline;
-  BOOL   caseless;
+  bool   multiline;
+  bool   caseless;
 } match_data;
 
 #define match_isgroup      true    /* Set if start of bracketed group */
@@ -166,7 +166,7 @@ Returns:     nothing
 */
 
 static void
-pchars(const UChar* p, int length, BOOL is_subject, match_data *md)
+pchars(const UChar* p, int length, bool is_subject, match_data *md)
 {
 int c;
 if (is_subject && length > md->end_subject - p) length = md->end_subject - p;
@@ -195,7 +195,7 @@ Arguments:
 Returns:      true if matched
 */
 
-static BOOL
+static bool
 match_ref(int offset, UChar* eptr, int length, match_data *md)
 {
 UChar* p = md->start_subject + md->offset_vector[offset];
@@ -411,11 +411,11 @@ static int match(UChar* eptr, const uschar* ecode, int offset_top, match_data* m
     
     unsigned rdepth = 0;
     
-    BOOL cur_is_word;
-    BOOL prev_is_word;
-    BOOL is_group_start = true;
+    bool cur_is_word;
+    bool prev_is_word;
+    bool is_group_start = true;
     int min;
-    BOOL minimize = false; /* Initialization not really needed, but some compilers think so. */
+    bool minimize = false; /* Initialization not really needed, but some compilers think so. */
     
     MatchStack stack;
 
@@ -2085,7 +2085,7 @@ int jsRegExpExecute(const JSRegExp* re,
     
     int ocount = offsetcount - (offsetcount % 3);
     
-    BOOL using_temporary_offsets = false;
+    bool using_temporary_offsets = false;
     if (re->top_backref > 0 && re->top_backref >= ocount/3) {
         ocount = re->top_backref * 3 + 3;
         match_block.offset_vector = new int[ocount];
@@ -2124,7 +2124,7 @@ int jsRegExpExecute(const JSRegExp* re,
      an unanchored pattern, of course. If there's no first char and the pattern was
      studied, there may be a bitmap of possible first characters. */
     
-    BOOL first_byte_caseless = false;
+    bool first_byte_caseless = false;
     int first_byte = -1;
     if (re->options & PCRE_FIRSTSET) {
         first_byte = re->first_byte & 255;
@@ -2135,7 +2135,7 @@ int jsRegExpExecute(const JSRegExp* re,
     /* For anchored or unanchored matches, there may be a "last known required
      character" set. */
     
-    BOOL req_byte_caseless = false;
+    bool req_byte_caseless = false;
     int req_byte = -1;
     int req_byte2 = -1;
     if (re->options & PCRE_REQCHSET) {
index b23d6016055d1d7e97926eecfb221e1075588484..ede039a1e31e9cd2cc4c857e7186100de0e24892 100644 (file)
@@ -292,12 +292,14 @@ static inline void getChar(int& c, const UChar* eptr)
 
 #define BACKCHAR(eptr) while(isTrailingSurrogate(*eptr)) eptr--;
 
-#define PCRE_FIRSTSET      0x40000000  /* first_byte is set */
-#define PCRE_REQCHSET      0x20000000  /* req_byte is set */
-#define PCRE_STARTLINE     0x10000000  /* start after \n for multiline */
-#define PCRE_ANCHORED      0x02000000  /* can't use partial with this regex */
-#define PCRE_CASELESS      0x00000001
-#define PCRE_MULTILINE     0x00000002
+enum PCREOptions {
+    PCRE_FIRSTSET = 0x40000000,  /* first_byte is set */
+    PCRE_REQCHSET = 0x20000000,  /* req_byte is set */
+    PCRE_STARTLINE = 0x10000000,  /* start after \n for multiline */
+    PCRE_ANCHORED = 0x02000000,  /* can't use partial with this regex */
+    PCRE_CASELESS = 0x00000001,
+    PCRE_MULTILINE = 0x00000002
+};
 
 /* Negative values for the firstchar and reqchar variables */
 
@@ -317,8 +319,6 @@ variable-length repeat, or a anything other than literal characters. */
 
 /* Miscellaneous definitions */
 
-typedef int BOOL;
-
 /* Flag bits and data types for the extended class (OP_XCLASS) for classes that
 contain UTF-8 characters with values greater than 255. */
 
@@ -562,9 +562,9 @@ typedef struct compile_data {
 one of the exported public functions. They have to be "external" in the C
 sense, but are not part of the PCRE public API. */
 
-extern int         _pcre_ord2utf8(int, uschar *);
+extern int         _pcre_ord2utf8(int, uschar*);
 extern int         _pcre_ucp_othercase(const unsigned int);
-extern BOOL        _pcre_xclass(int, const uschar *);
+extern bool        _pcre_xclass(int, const uschar*);
 
 static inline bool isNewline(UChar nl)
 {
index 7c1a61044f770cd9a23b476ae1ceffeaca4a5f6d..13946f3b20a4d0a2a76b61ae072079ed920c0d58 100644 (file)
@@ -73,9 +73,9 @@ static inline void getUTF8CharAndAdvancePointer(int& c, const uschar*& eptr)
     }
 }
 
-BOOL _pcre_xclass(int c, const uschar* data)
+bool _pcre_xclass(int c, const uschar* data)
 {
-    BOOL negated = (*data & XCL_NOT);
+    bool negated = (*data & XCL_NOT);
     
     /* Character values < 256 are matched against a bitmap, if one is present. If
      not, we still carry on, because there may be ranges that start below 256 in the