[Pcre-svn] [991] code/trunk: Rename the leave variable names…

Top Page
Delete this message
Author: Subversion repository
Date:  
To: pcre-svn
Subject: [Pcre-svn] [991] code/trunk: Rename the leave variable names to quit to improve WinCE compatibility.
Revision: 991
          http://vcs.pcre.org/viewvc?view=rev&revision=991
Author:   zherczeg
Date:     2012-07-08 17:44:39 +0100 (Sun, 08 Jul 2012)


Log Message:
-----------
Rename the leave variable names to quit to improve WinCE compatibility.

Modified Paths:
--------------
    code/trunk/ChangeLog
    code/trunk/pcre_jit_compile.c


Modified: code/trunk/ChangeLog
===================================================================
--- code/trunk/ChangeLog    2012-07-08 16:32:22 UTC (rev 990)
+++ code/trunk/ChangeLog    2012-07-08 16:44:39 UTC (rev 991)
@@ -14,7 +14,10 @@


4. Improved JIT compiler optimizations for character ranges.

+5.  Rename the "leave" variable names to "quit" to improve WinCE compatibility.
+    Reported by Giuseppe D'Angelo.


+
Version 8.31 06-July-2012
-------------------------


Modified: code/trunk/pcre_jit_compile.c
===================================================================
--- code/trunk/pcre_jit_compile.c    2012-07-08 16:32:22 UTC (rev 990)
+++ code/trunk/pcre_jit_compile.c    2012-07-08 16:44:39 UTC (rev 991)
@@ -181,7 +181,7 @@
   enum stub_types type;
   int data;
   struct sljit_jump *start;
-  struct sljit_label *leave;
+  struct sljit_label *quit;
   struct stub_list *next;
 } stub_list;


@@ -314,13 +314,13 @@

   /* Labels and jump lists. */
   struct sljit_label *partialmatchlabel;
-  struct sljit_label *leavelabel;
+  struct sljit_label *quitlabel;
   struct sljit_label *acceptlabel;
   stub_list *stubs;
   recurse_entry *entries;
   recurse_entry *currententry;
   jump_list *partialmatch;
-  jump_list *leave;
+  jump_list *quit;
   jump_list *accept;
   jump_list *calllimit;
   jump_list *stackalloc;
@@ -1680,7 +1680,7 @@
   list_item->type = type;
   list_item->data = data;
   list_item->start = start;
-  list_item->leave = LABEL();
+  list_item->quit = LABEL();
   list_item->next = common->stubs;
   common->stubs = list_item;
   }
@@ -1700,7 +1700,7 @@
     add_jump(compiler, &common->stackalloc, JUMP(SLJIT_FAST_CALL));
     break;
     }
-  JUMPTO(SLJIT_JUMP, list_item->leave);
+  JUMPTO(SLJIT_JUMP, list_item->quit);
   list_item = list_item->next;
   }
 common->stubs = NULL;
@@ -1810,7 +1810,7 @@
   OP1(SLJIT_MOV, SLJIT_RETURN_REG, 0, SLJIT_IMM, 1);
 }


-static SLJIT_INLINE void return_with_partial_match(compiler_common *common, struct sljit_label *leave)
+static SLJIT_INLINE void return_with_partial_match(compiler_common *common, struct sljit_label *quit)
{
DEFINE_COMPILER;

@@ -1820,7 +1820,7 @@
OP1(SLJIT_MOV, SLJIT_TEMPORARY_REG2, 0, ARGUMENTS, 0);
OP1(SLJIT_MOV, SLJIT_RETURN_REG, 0, SLJIT_IMM, PCRE_ERROR_PARTIAL);
OP1(SLJIT_MOV_SI, SLJIT_TEMPORARY_REG3, 0, SLJIT_MEM1(SLJIT_TEMPORARY_REG2), SLJIT_OFFSETOF(jit_arguments, offsetcount));
-CMPTO(SLJIT_C_LESS, SLJIT_TEMPORARY_REG3, 0, SLJIT_IMM, 2, leave);
+CMPTO(SLJIT_C_LESS, SLJIT_TEMPORARY_REG3, 0, SLJIT_IMM, 2, quit);

/* Store match begin and end. */
OP1(SLJIT_MOV, SLJIT_SAVED_REG1, 0, SLJIT_MEM1(SLJIT_TEMPORARY_REG2), SLJIT_OFFSETOF(jit_arguments, begin));
@@ -1838,7 +1838,7 @@
#endif
OP1(SLJIT_MOV_SI, SLJIT_MEM1(SLJIT_TEMPORARY_REG2), 0, SLJIT_TEMPORARY_REG3, 0);

-JUMPTO(SLJIT_JUMP, leave);
+JUMPTO(SLJIT_JUMP, quit);
}

static SLJIT_INLINE void check_start_used_ptr(compiler_common *common)
@@ -2518,7 +2518,7 @@
{
DEFINE_COMPILER;
struct sljit_label *start;
-struct sljit_jump *leave;
+struct sljit_jump *quit;
struct sljit_jump *found;
pcre_int32 chars[4];
pcre_uchar *cc = common->start + 1 + IMM2_SIZE;
@@ -2654,7 +2654,7 @@
OP2(SLJIT_SUB, STR_END, 0, STR_END, 0, SLJIT_IMM, 1);

start = LABEL();
-leave = CMP(SLJIT_C_GREATER_EQUAL, STR_PTR, 0, STR_END, 0);
+quit = CMP(SLJIT_C_GREATER_EQUAL, STR_PTR, 0, STR_END, 0);
#if defined SLJIT_UNALIGNED && SLJIT_UNALIGNED
#ifdef COMPILE_PCRE8
OP1(SLJIT_MOV_UH, TMP1, 0, SLJIT_MEM1(STR_PTR), 0);
@@ -2695,7 +2695,7 @@
OP2(SLJIT_ADD, STR_PTR, 0, STR_PTR, 0, SLJIT_IMM, IN_UCHARS(1));
JUMPTO(SLJIT_JUMP, start);
JUMPHERE(found);
-JUMPHERE(leave);
+JUMPHERE(quit);

if (firstline)
OP1(SLJIT_MOV, STR_END, 0, SLJIT_MEM1(SLJIT_LOCALS_REG), POSSESSIVE0);
@@ -2707,7 +2707,7 @@
{
DEFINE_COMPILER;
struct sljit_label *start;
-struct sljit_jump *leave;
+struct sljit_jump *quit;
struct sljit_jump *found;
pcre_uchar oc, bit;

@@ -2718,7 +2718,7 @@
}

start = LABEL();
-leave = CMP(SLJIT_C_GREATER_EQUAL, STR_PTR, 0, STR_END, 0);
+quit = CMP(SLJIT_C_GREATER_EQUAL, STR_PTR, 0, STR_END, 0);
OP1(MOV_UCHAR, TMP1, 0, SLJIT_MEM1(STR_PTR), 0);

oc = first_char;
@@ -2753,7 +2753,7 @@
OP2(SLJIT_ADD, STR_PTR, 0, STR_PTR, 0, SLJIT_IMM, IN_UCHARS(1));
JUMPTO(SLJIT_JUMP, start);
JUMPHERE(found);
-JUMPHERE(leave);
+JUMPHERE(quit);

if (firstline)
OP1(SLJIT_MOV, STR_END, 0, SLJIT_MEM1(SLJIT_LOCALS_REG), POSSESSIVE0);
@@ -2765,7 +2765,7 @@
struct sljit_label *loop;
struct sljit_jump *lastchar;
struct sljit_jump *firstchar;
-struct sljit_jump *leave;
+struct sljit_jump *quit;
struct sljit_jump *foundcr = NULL;
struct sljit_jump *notfoundnl;
jump_list *newline = NULL;
@@ -2794,13 +2794,13 @@

loop = LABEL();
OP2(SLJIT_ADD, STR_PTR, 0, STR_PTR, 0, SLJIT_IMM, IN_UCHARS(1));
- leave = CMP(SLJIT_C_GREATER_EQUAL, STR_PTR, 0, STR_END, 0);
+ quit = CMP(SLJIT_C_GREATER_EQUAL, STR_PTR, 0, STR_END, 0);
OP1(MOV_UCHAR, TMP1, 0, SLJIT_MEM1(STR_PTR), IN_UCHARS(-2));
OP1(MOV_UCHAR, TMP2, 0, SLJIT_MEM1(STR_PTR), IN_UCHARS(-1));
CMPTO(SLJIT_C_NOT_EQUAL, TMP1, 0, SLJIT_IMM, (common->newline >> 8) & 0xff, loop);
CMPTO(SLJIT_C_NOT_EQUAL, TMP2, 0, SLJIT_IMM, common->newline & 0xff, loop);

- JUMPHERE(leave);
+ JUMPHERE(quit);
JUMPHERE(firstchar);
JUMPHERE(lastchar);

@@ -2824,7 +2824,7 @@

if (common->nltype == NLTYPE_ANY || common->nltype == NLTYPE_ANYCRLF)
{
- leave = JUMP(SLJIT_JUMP);
+ quit = JUMP(SLJIT_JUMP);
JUMPHERE(foundcr);
notfoundnl = CMP(SLJIT_C_GREATER_EQUAL, STR_PTR, 0, STR_END, 0);
OP1(MOV_UCHAR, TMP1, 0, SLJIT_MEM1(STR_PTR), 0);
@@ -2835,7 +2835,7 @@
#endif
OP2(SLJIT_ADD, STR_PTR, 0, STR_PTR, 0, TMP1, 0);
JUMPHERE(notfoundnl);
- JUMPHERE(leave);
+ JUMPHERE(quit);
}
JUMPHERE(lastchar);
JUMPHERE(firstchar);
@@ -2848,7 +2848,7 @@
{
DEFINE_COMPILER;
struct sljit_label *start;
-struct sljit_jump *leave;
+struct sljit_jump *quit;
struct sljit_jump *found;
#ifndef COMPILE_PCRE8
struct sljit_jump *jump;
@@ -2861,7 +2861,7 @@
}

start = LABEL();
-leave = CMP(SLJIT_C_GREATER_EQUAL, STR_PTR, 0, STR_END, 0);
+quit = CMP(SLJIT_C_GREATER_EQUAL, STR_PTR, 0, STR_END, 0);
OP1(MOV_UCHAR, TMP1, 0, SLJIT_MEM1(STR_PTR), 0);
#ifdef SUPPORT_UTF
if (common->utf)
@@ -2905,7 +2905,7 @@
#endif
JUMPTO(SLJIT_JUMP, start);
JUMPHERE(found);
-JUMPHERE(leave);
+JUMPHERE(quit);

if (firstline)
OP1(SLJIT_MOV, STR_END, 0, SLJIT_MEM1(SLJIT_LOCALS_REG), POSSESSIVE0);
@@ -4940,9 +4940,9 @@
jump_list **target = (conditional) ? &backtrack->condfailed : &backtrack->common.topbacktracks;
jump_list **found;
/* Saving previous accept variables. */
-struct sljit_label *save_leavelabel = common->leavelabel;
+struct sljit_label *save_quitlabel = common->quitlabel;
struct sljit_label *save_acceptlabel = common->acceptlabel;
-jump_list *save_leave = common->leave;
+jump_list *save_quit = common->quit;
jump_list *save_accept = common->accept;
struct sljit_jump *jump;
struct sljit_jump *brajump = NULL;
@@ -4990,8 +4990,8 @@
}

 memset(&altbacktrack, 0, sizeof(backtrack_common));
-common->leavelabel = NULL;
-common->leave = NULL;
+common->quitlabel = NULL;
+common->quit = NULL;
 while (1)
   {
   common->acceptlabel = NULL;
@@ -5006,9 +5006,9 @@
   compile_trypath(common, ccbegin + 1 + LINK_SIZE, cc, &altbacktrack);
   if (SLJIT_UNLIKELY(sljit_get_compiler_error(compiler)))
     {
-    common->leavelabel = save_leavelabel;
+    common->quitlabel = save_quitlabel;
     common->acceptlabel = save_acceptlabel;
-    common->leave = save_leave;
+    common->quit = save_quit;
     common->accept = save_accept;
     return NULL;
     }
@@ -5061,9 +5061,9 @@
   compile_backtrackpath(common, altbacktrack.top);
   if (SLJIT_UNLIKELY(sljit_get_compiler_error(compiler)))
     {
-    common->leavelabel = save_leavelabel;
+    common->quitlabel = save_quitlabel;
     common->acceptlabel = save_acceptlabel;
-    common->leave = save_leave;
+    common->quit = save_quit;
     common->accept = save_accept;
     return NULL;
     }
@@ -5076,8 +5076,8 @@
   cc += GET(cc, 1);
   }
 /* None of them matched. */
-if (common->leave != NULL)
-  set_jumps(common->leave, LABEL());
+if (common->quit != NULL)
+  set_jumps(common->quit, LABEL());


 if (opcode == OP_ASSERT || opcode == OP_ASSERTBACK)
   {
@@ -5202,9 +5202,9 @@
     }
   }


-common->leavelabel = save_leavelabel;
+common->quitlabel = save_quitlabel;
common->acceptlabel = save_acceptlabel;
-common->leave = save_leave;
+common->quit = save_quit;
common->accept = save_accept;
return cc + 1 + LINK_SIZE;
}
@@ -7502,10 +7502,10 @@

     case OP_COMMIT:
     OP1(SLJIT_MOV, SLJIT_RETURN_REG, 0, SLJIT_IMM, PCRE_ERROR_NOMATCH);
-    if (common->leavelabel == NULL)
-      add_jump(compiler, &common->leave, JUMP(SLJIT_JUMP));
+    if (common->quitlabel == NULL)
+      add_jump(compiler, &common->quit, JUMP(SLJIT_JUMP));
     else
-      JUMPTO(SLJIT_JUMP, common->leavelabel);
+      JUMPTO(SLJIT_JUMP, common->quitlabel);
     break;


     case OP_FAIL:
@@ -7533,8 +7533,8 @@
 int alternativesize;
 BOOL needsframe;
 backtrack_common altbacktrack;
-struct sljit_label *save_leavelabel = common->leavelabel;
-jump_list *save_leave = common->leave;
+struct sljit_label *save_quitlabel = common->quitlabel;
+jump_list *save_quit = common->quit;
 struct sljit_jump *jump;


SLJIT_ASSERT(*cc == OP_BRA || *cc == OP_CBRA || *cc == OP_CBRAPOS || *cc == OP_SCBRA || *cc == OP_SCBRAPOS);
@@ -7559,9 +7559,9 @@
OP1(SLJIT_MOV, SLJIT_MEM1(STACK_TOP), STACK(0), STR_PTR, 0);

 memset(&altbacktrack, 0, sizeof(backtrack_common));
-common->leavelabel = NULL;
+common->quitlabel = NULL;
 common->acceptlabel = NULL;
-common->leave = NULL;
+common->quit = NULL;
 common->accept = NULL;
 altbacktrack.cc = ccbegin;
 cc += GET(cc, 1);
@@ -7576,8 +7576,8 @@
   compile_trypath(common, altbacktrack.cc, cc, &altbacktrack);
   if (SLJIT_UNLIKELY(sljit_get_compiler_error(compiler)))
     {
-    common->leavelabel = save_leavelabel;
-    common->leave = save_leave;
+    common->quitlabel = save_quitlabel;
+    common->quit = save_quit;
     return;
     }


@@ -7586,8 +7586,8 @@
   compile_backtrackpath(common, altbacktrack.top);
   if (SLJIT_UNLIKELY(sljit_get_compiler_error(compiler)))
     {
-    common->leavelabel = save_leavelabel;
-    common->leave = save_leave;
+    common->quitlabel = save_quitlabel;
+    common->quit = save_quit;
     return;
     }
   set_jumps(altbacktrack.topbacktracks, LABEL());
@@ -7599,8 +7599,8 @@
   cc += GET(cc, 1);
   }
 /* None of them matched. */
-if (common->leave != NULL)
-  set_jumps(common->leave, LABEL());
+if (common->quit != NULL)
+  set_jumps(common->quit, LABEL());


OP1(SLJIT_MOV, TMP3, 0, SLJIT_IMM, 0);
jump = JUMP(SLJIT_JUMP);
@@ -7623,8 +7623,8 @@
OP1(SLJIT_MOV, SLJIT_MEM1(SLJIT_LOCALS_REG), common->recursive_head, TMP2, 0);
sljit_emit_fast_return(compiler, SLJIT_MEM1(STACK_TOP), 0);

-common->leavelabel = save_leavelabel;
-common->leave = save_leave;
+common->quitlabel = save_quitlabel;
+common->quit = save_quit;
}

#undef COMPILE_BACKTRACKPATH
@@ -7840,16 +7840,16 @@

/* This means we have a match. Update the ovector. */
copy_ovector(common, re->top_bracket + 1);
-common->leavelabel = LABEL();
-if (common->leave != NULL)
- set_jumps(common->leave, common->leavelabel);
+common->quitlabel = LABEL();
+if (common->quit != NULL)
+ set_jumps(common->quit, common->quitlabel);
sljit_emit_return(compiler, SLJIT_MOV, SLJIT_RETURN_REG, 0);

if (mode != JIT_COMPILE)
{
common->partialmatchlabel = LABEL();
set_jumps(common->partialmatch, common->partialmatchlabel);
- return_with_partial_match(common, common->leavelabel);
+ return_with_partial_match(common, common->quitlabel);
}

empty_match_backtrack = LABEL();
@@ -7913,7 +7913,7 @@
CMPTO(SLJIT_C_NOT_EQUAL, SLJIT_MEM1(SLJIT_LOCALS_REG), common->hit_start, SLJIT_IMM, 0, common->partialmatchlabel);

OP1(SLJIT_MOV, SLJIT_RETURN_REG, 0, SLJIT_IMM, PCRE_ERROR_NOMATCH);
-JUMPTO(SLJIT_JUMP, common->leavelabel);
+JUMPTO(SLJIT_JUMP, common->quitlabel);

flush_stubs(common);

@@ -7966,12 +7966,12 @@
JUMPHERE(jump);
/* We break the return address cache here, but this is a really rare case. */
OP1(SLJIT_MOV, SLJIT_RETURN_REG, 0, SLJIT_IMM, PCRE_ERROR_JIT_STACKLIMIT);
-JUMPTO(SLJIT_JUMP, common->leavelabel);
+JUMPTO(SLJIT_JUMP, common->quitlabel);

/* Call limit reached. */
set_jumps(common->calllimit, LABEL());
OP1(SLJIT_MOV, SLJIT_RETURN_REG, 0, SLJIT_IMM, PCRE_ERROR_MATCHLIMIT);
-JUMPTO(SLJIT_JUMP, common->leavelabel);
+JUMPTO(SLJIT_JUMP, common->quitlabel);

if (common->revertframes != NULL)
{