]> git.ipfire.org Git - thirdparty/postgresql.git/commitdiff
Sync our regex code with upstream changes since last time we did this, which
authorTom Lane <tgl@sss.pgh.pa.us>
Thu, 14 Feb 2008 17:33:37 +0000 (17:33 +0000)
committerTom Lane <tgl@sss.pgh.pa.us>
Thu, 14 Feb 2008 17:33:37 +0000 (17:33 +0000)
was Tcl 8.4.8.  The main changes are to remove the never-fully-implemented
code for multi-character collating elements, and to const-ify some stuff a
bit more fully.  In combination with the recent security patch, this commit
brings us into line with Tcl 8.5.0.

Note that I didn't make any effort to duplicate a lot of cosmetic changes
that they made to bring their copy into line with their own style
guidelines, such as adding braces around single-line IF bodies.  Most of
those we either had done already (such as ANSI-fication of function headers)
or there is no point because pgindent would undo the change anyway.

src/backend/regex/regc_color.c
src/backend/regex/regc_cvec.c
src/backend/regex/regc_lex.c
src/backend/regex/regc_locale.c
src/backend/regex/regc_nfa.c
src/backend/regex/regcomp.c
src/backend/regex/regerror.c
src/include/regex/regcustom.h
src/include/regex/regguts.h

index 87eb1e4958a28db84a0470cb98847dd9960adf04..e15fd4b788e9f1af723bece15431909b40770112 100644 (file)
@@ -28,7 +28,7 @@
  * OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF
  * ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  *
- * $PostgreSQL: pgsql/src/backend/regex/regc_color.c,v 1.8 2008/01/03 20:47:55 tgl Exp $
+ * $PostgreSQL: pgsql/src/backend/regex/regc_color.c,v 1.9 2008/02/14 17:33:37 tgl Exp $
  *
  *
  * Note that there are some incestuous relationships between this code and
@@ -222,7 +222,6 @@ static color                                        /* COLORLESS for error */
 newcolor(struct colormap * cm)
 {
        struct colordesc *cd;
-       struct colordesc *new;
        size_t          n;
 
        if (CISERR())
@@ -245,24 +244,25 @@ newcolor(struct colormap * cm)
        else
        {
                /* oops, must allocate more */
+               struct colordesc *newCd;
+
                n = cm->ncds * 2;
                if (cm->cd == cm->cdspace)
                {
-                       new = (struct colordesc *) MALLOC(n *
-                                                                                         sizeof(struct colordesc));
-                       if (new != NULL)
-                               memcpy(VS(new), VS(cm->cdspace), cm->ncds *
+                       newCd = (struct colordesc *) MALLOC(n * sizeof(struct colordesc));
+                       if (newCd != NULL)
+                               memcpy(VS(newCd), VS(cm->cdspace), cm->ncds *
                                           sizeof(struct colordesc));
                }
                else
-                       new = (struct colordesc *) REALLOC(cm->cd,
-                                                                                          n * sizeof(struct colordesc));
-               if (new == NULL)
+                       newCd = (struct colordesc *)
+                               REALLOC(cm->cd, n * sizeof(struct colordesc));
+               if (newCd == NULL)
                {
                        CERR(REG_ESPACE);
                        return COLORLESS;
                }
-               cm->cd = new;
+               cm->cd = newCd;
                cm->ncds = n;
                assert(cm->max < cm->ncds - 1);
                cm->max++;
@@ -634,21 +634,6 @@ uncolorchain(struct colormap * cm,
        a->colorchainRev = NULL;
 }
 
-/*
- * singleton - is this character in its own color?
- */
-static int                                             /* predicate */
-singleton(struct colormap * cm,
-                 chr c)
-{
-       color           co;                             /* color of c */
-
-       co = GETCOLOR(cm, c);
-       if (cm->cd[co].nchrs == 1 && cm->cd[co].sub == NOSUB)
-               return 1;
-       return 0;
-}
-
 /*
  * rainbow - add arcs of all full colors (but one) between specified states
  */
index 719c4c5ef3b51478f3e63c03ca51963617b2c4f1..25bfae3e3284dc60b8abfdcbcc5912411a6ad9fc 100644 (file)
  * OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF
  * ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  *
- * $PostgreSQL: pgsql/src/backend/regex/regc_cvec.c,v 1.5 2005/10/15 02:49:24 momjian Exp $
+ * $PostgreSQL: pgsql/src/backend/regex/regc_cvec.c,v 1.6 2008/02/14 17:33:37 tgl Exp $
  *
  */
 
+/*
+ * Notes:
+ * Only (selected) functions in _this_ file should treat chr* as non-constant.
+ */
+
 /*
  * newcvec - allocate a new cvec
  */
 static struct cvec *
 newcvec(int nchrs,                             /* to hold this many chrs... */
-               int nranges,                    /* ... and this many ranges... */
-               int nmcces)                             /* ... and this many MCCEs */
+               int nranges)                    /* ... and this many ranges */
 {
-       size_t          n;
-       size_t          nc;
-       struct cvec *cv;
+       size_t          nc = (size_t) nchrs + (size_t) nranges * 2;
+       size_t          n = sizeof(struct cvec) + nc * sizeof(chr);
+       struct cvec *cv = (struct cvec *) MALLOC(n);
 
-       nc = (size_t) nchrs + (size_t) nmcces *(MAXMCCE + 1) + (size_t) nranges *2;
-
-       n = sizeof(struct cvec) + (size_t) (nmcces - 1) * sizeof(chr *)
-               + nc * sizeof(chr);
-       cv = (struct cvec *) MALLOC(n);
        if (cv == NULL)
                return NULL;
        cv->chrspace = nchrs;
-       cv->chrs = (chr *) &cv->mcces[nmcces];          /* chrs just after MCCE ptrs */
-       cv->mccespace = nmcces;
-       cv->ranges = cv->chrs + nchrs + nmcces * (MAXMCCE + 1);
+       cv->chrs = (chr *) (((char *) cv) + sizeof(struct cvec));
+       cv->ranges = cv->chrs + nchrs;
        cv->rangespace = nranges;
        return clearcvec(cv);
 }
@@ -66,17 +64,9 @@ newcvec(int nchrs,                           /* to hold this many chrs... */
 static struct cvec *
 clearcvec(struct cvec * cv)
 {
-       int                     i;
-
        assert(cv != NULL);
        cv->nchrs = 0;
-       assert(cv->chrs == (chr *) &cv->mcces[cv->mccespace]);
-       cv->nmcces = 0;
-       cv->nmccechrs = 0;
        cv->nranges = 0;
-       for (i = 0; i < cv->mccespace; i++)
-               cv->mcces[i] = NULL;
-
        return cv;
 }
 
@@ -87,7 +77,6 @@ static void
 addchr(struct cvec * cv,               /* character vector */
           chr c)                                       /* character to add */
 {
-       assert(cv->nchrs < cv->chrspace - cv->nmccechrs);
        cv->chrs[cv->nchrs++] = (chr) c;
 }
 
@@ -105,73 +94,21 @@ addrange(struct cvec * cv,         /* character vector */
        cv->nranges++;
 }
 
-/*
- * addmcce - add an MCCE to a cvec
- */
-static void
-addmcce(struct cvec * cv,              /* character vector */
-               chr *startp,                    /* beginning of text */
-               chr *endp)                              /* just past end of text */
-{
-       int                     len;
-       int                     i;
-       chr                *s;
-       chr                *d;
-
-       if (startp == NULL && endp == NULL)
-               return;
-       len = endp - startp;
-       assert(len > 0);
-       assert(cv->nchrs + len < cv->chrspace - cv->nmccechrs);
-       assert(cv->nmcces < cv->mccespace);
-       d = &cv->chrs[cv->chrspace - cv->nmccechrs - len - 1];
-       cv->mcces[cv->nmcces++] = d;
-       for (s = startp, i = len; i > 0; s++, i--)
-               *d++ = *s;
-       *d++ = 0;                                       /* endmarker */
-       assert(d == &cv->chrs[cv->chrspace - cv->nmccechrs]);
-       cv->nmccechrs += len + 1;
-}
-
-/*
- * haschr - does a cvec contain this chr?
- */
-static int                                             /* predicate */
-haschr(struct cvec * cv,               /* character vector */
-          chr c)                                       /* character to test for */
-{
-       int                     i;
-       chr                *p;
-
-       for (p = cv->chrs, i = cv->nchrs; i > 0; p++, i--)
-       {
-               if (*p == c)
-                       return 1;
-       }
-       for (p = cv->ranges, i = cv->nranges; i > 0; p += 2, i--)
-       {
-               if ((*p <= c) && (c <= *(p + 1)))
-                       return 1;
-       }
-       return 0;
-}
-
 /*
  * getcvec - get a cvec, remembering it as v->cv
  */
 static struct cvec *
 getcvec(struct vars * v,               /* context */
                int nchrs,                              /* to hold this many chrs... */
-               int nranges,                    /* ... and this many ranges... */
-               int nmcces)                             /* ... and this many MCCEs */
+               int nranges)                    /* ... and this many ranges */
 {
        if (v->cv != NULL && nchrs <= v->cv->chrspace &&
-               nranges <= v->cv->rangespace && nmcces <= v->cv->mccespace)
+               nranges <= v->cv->rangespace)
                return clearcvec(v->cv);
 
        if (v->cv != NULL)
                freecvec(v->cv);
-       v->cv = newcvec(nchrs, nranges, nmcces);
+       v->cv = newcvec(nchrs, nranges);
        if (v->cv == NULL)
                ERR(REG_ESPACE);
 
index fc86ca322a902a07ef01b3c2937f1936acfbb922..d2fc591ed8ab730bcc78cd02897eb7a4f76d1fb9 100644 (file)
@@ -28,7 +28,7 @@
  * OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF
  * ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  *
- * $PostgreSQL: pgsql/src/backend/regex/regc_lex.c,v 1.7 2008/01/03 20:47:55 tgl Exp $
+ * $PostgreSQL: pgsql/src/backend/regex/regc_lex.c,v 1.8 2008/02/14 17:33:37 tgl Exp $
  *
  */
 
@@ -201,8 +201,8 @@ prefixes(struct vars * v)
  */
 static void
 lexnest(struct vars * v,
-               chr *beginp,                    /* start of interpolation */
-               chr *endp)                              /* one past end of interpolation */
+               const chr *beginp,              /* start of interpolation */
+               const chr *endp)                /* one past end of interpolation */
 {
        assert(v->savenow == NULL); /* only one level of nesting */
        v->savenow = v->now;
@@ -214,47 +214,47 @@ lexnest(struct vars * v,
 /*
  * string constants to interpolate as expansions of things like \d
  */
-static chr     backd[] = {                     /* \d */
+static const chr backd[] = {                   /* \d */
        CHR('['), CHR('['), CHR(':'),
        CHR('d'), CHR('i'), CHR('g'), CHR('i'), CHR('t'),
        CHR(':'), CHR(']'), CHR(']')
 };
-static chr     backD[] = {                     /* \D */
+static const chr backD[] = {                   /* \D */
        CHR('['), CHR('^'), CHR('['), CHR(':'),
        CHR('d'), CHR('i'), CHR('g'), CHR('i'), CHR('t'),
        CHR(':'), CHR(']'), CHR(']')
 };
-static chr     brbackd[] = {           /* \d within brackets */
+static const chr brbackd[] = {                 /* \d within brackets */
        CHR('['), CHR(':'),
        CHR('d'), CHR('i'), CHR('g'), CHR('i'), CHR('t'),
        CHR(':'), CHR(']')
 };
-static chr     backs[] = {                     /* \s */
+static const chr backs[] = {                   /* \s */
        CHR('['), CHR('['), CHR(':'),
        CHR('s'), CHR('p'), CHR('a'), CHR('c'), CHR('e'),
        CHR(':'), CHR(']'), CHR(']')
 };
-static chr     backS[] = {                     /* \S */
+static const chr backS[] = {                   /* \S */
        CHR('['), CHR('^'), CHR('['), CHR(':'),
        CHR('s'), CHR('p'), CHR('a'), CHR('c'), CHR('e'),
        CHR(':'), CHR(']'), CHR(']')
 };
-static chr     brbacks[] = {           /* \s within brackets */
+static const chr brbacks[] = {                 /* \s within brackets */
        CHR('['), CHR(':'),
        CHR('s'), CHR('p'), CHR('a'), CHR('c'), CHR('e'),
        CHR(':'), CHR(']')
 };
-static chr     backw[] = {                     /* \w */
+static const chr backw[] = {                   /* \w */
        CHR('['), CHR('['), CHR(':'),
        CHR('a'), CHR('l'), CHR('n'), CHR('u'), CHR('m'),
        CHR(':'), CHR(']'), CHR('_'), CHR(']')
 };
-static chr     backW[] = {                     /* \W */
+static const chr backW[] = {                   /* \W */
        CHR('['), CHR('^'), CHR('['), CHR(':'),
        CHR('a'), CHR('l'), CHR('n'), CHR('u'), CHR('m'),
        CHR(':'), CHR(']'), CHR('_'), CHR(']')
 };
-static chr     brbackw[] = {           /* \w within brackets */
+static const chr brbackw[] = {                 /* \w within brackets */
        CHR('['), CHR(':'),
        CHR('a'), CHR('l'), CHR('n'), CHR('u'), CHR('m'),
        CHR(':'), CHR(']'), CHR('_')
@@ -722,7 +722,7 @@ lexescape(struct vars * v)
        static chr      esc[] = {
                CHR('E'), CHR('S'), CHR('C')
        };
-       chr                *save;
+       const chr  *save;
 
        assert(v->cflags & REG_ADVF);
 
@@ -1080,7 +1080,7 @@ brenext(struct vars * v,
 static void
 skip(struct vars * v)
 {
-       chr                *start = v->now;
+       const chr  *start = v->now;
 
        assert(v->cflags & REG_EXPANDED);
 
@@ -1119,8 +1119,8 @@ newline(void)
  */
 static chr
 chrnamed(struct vars * v,
-                chr *startp,                   /* start of name */
-                chr *endp,                             /* just past end of name */
+                const chr *startp,             /* start of name */
+                const chr *endp,               /* just past end of name */
                 chr lastresort)                /* what to return if name lookup fails */
 {
        celt            c;
index 09c1bd127da336500ac1c6486c797c965ce3df35..28f5e7ca12b02106ccaadcb2e096fbb28db502f9 100644 (file)
  * permission to use and distribute the software in accordance with the
  * terms specified in this license.
  *
- * $PostgreSQL: pgsql/src/backend/regex/regc_locale.c,v 1.8 2005/11/22 18:17:19 momjian Exp $
+ * $PostgreSQL: pgsql/src/backend/regex/regc_locale.c,v 1.9 2008/02/14 17:33:37 tgl Exp $
  */
 
 /* ASCII character-name table */
 
-static struct cname
+static const struct cname
 {
-       char       *name;
-       char            code;
+       const char *name;
+       const char      code;
 }      cnames[] =
 
 {
@@ -423,46 +423,15 @@ pg_wc_tolower(pg_wchar c)
 }
 
 
-/*
- * nmcces - how many distinct MCCEs are there?
- */
-static int
-nmcces(struct vars * v)
-{
-       /*
-        * No multi-character collating elements defined at the moment.
-        */
-       return 0;
-}
-
-/*
- * nleaders - how many chrs can be first chrs of MCCEs?
- */
-static int
-nleaders(struct vars * v)
-{
-       return 0;
-}
-
-/*
- * allmcces - return a cvec with all the MCCEs of the locale
- */
-static struct cvec *
-allmcces(struct vars * v,              /* context */
-                struct cvec * cv)              /* this is supposed to have enough room */
-{
-       return clearcvec(cv);
-}
-
 /*
  * element - map collating-element name to celt
  */
 static celt
 element(struct vars * v,               /* context */
-               chr *startp,                    /* points to start of name */
-               chr *endp)                              /* points just past end of name */
+               const chr *startp,              /* points to start of name */
+               const chr *endp)                /* points just past end of name */
 {
-       struct cname *cn;
+       const struct cname *cn;
        size_t          len;
 
        /* generic:  one-chr names stand for themselves */
@@ -513,7 +482,7 @@ range(struct vars * v,                      /* context */
 
        if (!cases)
        {                                                       /* easy version */
-               cv = getcvec(v, 0, 1, 0);
+               cv = getcvec(v, 0, 1);
                NOERRN();
                addrange(cv, a, b);
                return cv;
@@ -527,7 +496,7 @@ range(struct vars * v,                      /* context */
 
        nchrs = (b - a + 1) * 2 + 4;
 
-       cv = getcvec(v, nchrs, 0, 0);
+       cv = getcvec(v, nchrs, 0);
        NOERRN();
 
        for (c = a; c <= b; c++)
@@ -550,7 +519,6 @@ range(struct vars * v,                      /* context */
 static int                                             /* predicate */
 before(celt x, celt y)
 {
-       /* trivial because no MCCEs */
        if (x < y)
                return 1;
        return 0;
@@ -571,7 +539,7 @@ eclass(struct vars * v,                     /* context */
        /* crude fake equivalence class for testing */
        if ((v->cflags & REG_FAKE) && c == 'x')
        {
-               cv = getcvec(v, 4, 0, 0);
+               cv = getcvec(v, 4, 0);
                addchr(cv, (chr) 'x');
                addchr(cv, (chr) 'y');
                if (cases)
@@ -585,7 +553,7 @@ eclass(struct vars * v,                     /* context */
        /* otherwise, none */
        if (cases)
                return allcases(v, c);
-       cv = getcvec(v, 1, 0, 0);
+       cv = getcvec(v, 1, 0);
        assert(cv != NULL);
        addchr(cv, (chr) c);
        return cv;
@@ -598,13 +566,13 @@ eclass(struct vars * v,                   /* context */
  */
 static struct cvec *
 cclass(struct vars * v,                        /* context */
-          chr *startp,                         /* where the name starts */
-          chr *endp,                           /* just past the end of the name */
+          const chr *startp,           /* where the name starts */
+          const chr *endp,                     /* just past the end of the name */
           int cases)                           /* case-independent? */
 {
        size_t          len;
        struct cvec *cv = NULL;
-       char      **namePtr;
+       const char **namePtr;
        int                     i,
                                index;
 
@@ -612,7 +580,7 @@ cclass(struct vars * v,                     /* context */
         * The following arrays define the valid character class names.
         */
 
-       static char *classNames[] = {
+       static const char *classNames[] = {
                "alnum", "alpha", "ascii", "blank", "cntrl", "digit", "graph",
                "lower", "print", "punct", "space", "upper", "xdigit", NULL
        };
@@ -662,7 +630,7 @@ cclass(struct vars * v,                     /* context */
        switch ((enum classes) index)
        {
                case CC_PRINT:
-                       cv = getcvec(v, UCHAR_MAX, 0, 0);
+                       cv = getcvec(v, UCHAR_MAX, 0);
                        if (cv)
                        {
                                for (i = 0; i <= UCHAR_MAX; i++)
@@ -673,7 +641,7 @@ cclass(struct vars * v,                     /* context */
                        }
                        break;
                case CC_ALNUM:
-                       cv = getcvec(v, UCHAR_MAX, 0, 0);
+                       cv = getcvec(v, UCHAR_MAX, 0);
                        if (cv)
                        {
                                for (i = 0; i <= UCHAR_MAX; i++)
@@ -684,7 +652,7 @@ cclass(struct vars * v,                     /* context */
                        }
                        break;
                case CC_ALPHA:
-                       cv = getcvec(v, UCHAR_MAX, 0, 0);
+                       cv = getcvec(v, UCHAR_MAX, 0);
                        if (cv)
                        {
                                for (i = 0; i <= UCHAR_MAX; i++)
@@ -695,27 +663,27 @@ cclass(struct vars * v,                   /* context */
                        }
                        break;
                case CC_ASCII:
-                       cv = getcvec(v, 0, 1, 0);
+                       cv = getcvec(v, 0, 1);
                        if (cv)
                                addrange(cv, 0, 0x7f);
                        break;
                case CC_BLANK:
-                       cv = getcvec(v, 2, 0, 0);
+                       cv = getcvec(v, 2, 0);
                        addchr(cv, '\t');
                        addchr(cv, ' ');
                        break;
                case CC_CNTRL:
-                       cv = getcvec(v, 0, 2, 0);
+                       cv = getcvec(v, 0, 2);
                        addrange(cv, 0x0, 0x1f);
                        addrange(cv, 0x7f, 0x9f);
                        break;
                case CC_DIGIT:
-                       cv = getcvec(v, 0, 1, 0);
+                       cv = getcvec(v, 0, 1);
                        if (cv)
                                addrange(cv, (chr) '0', (chr) '9');
                        break;
                case CC_PUNCT:
-                       cv = getcvec(v, UCHAR_MAX, 0, 0);
+                       cv = getcvec(v, UCHAR_MAX, 0);
                        if (cv)
                        {
                                for (i = 0; i <= UCHAR_MAX; i++)
@@ -726,7 +694,7 @@ cclass(struct vars * v,                     /* context */
                        }
                        break;
                case CC_XDIGIT:
-                       cv = getcvec(v, 0, 3, 0);
+                       cv = getcvec(v, 0, 3);
                        if (cv)
                        {
                                addrange(cv, '0', '9');
@@ -735,7 +703,7 @@ cclass(struct vars * v,                     /* context */
                        }
                        break;
                case CC_SPACE:
-                       cv = getcvec(v, UCHAR_MAX, 0, 0);
+                       cv = getcvec(v, UCHAR_MAX, 0);
                        if (cv)
                        {
                                for (i = 0; i <= UCHAR_MAX; i++)
@@ -746,7 +714,7 @@ cclass(struct vars * v,                     /* context */
                        }
                        break;
                case CC_LOWER:
-                       cv = getcvec(v, UCHAR_MAX, 0, 0);
+                       cv = getcvec(v, UCHAR_MAX, 0);
                        if (cv)
                        {
                                for (i = 0; i <= UCHAR_MAX; i++)
@@ -757,7 +725,7 @@ cclass(struct vars * v,                     /* context */
                        }
                        break;
                case CC_UPPER:
-                       cv = getcvec(v, UCHAR_MAX, 0, 0);
+                       cv = getcvec(v, UCHAR_MAX, 0);
                        if (cv)
                        {
                                for (i = 0; i <= UCHAR_MAX; i++)
@@ -768,7 +736,7 @@ cclass(struct vars * v,                     /* context */
                        }
                        break;
                case CC_GRAPH:
-                       cv = getcvec(v, UCHAR_MAX, 0, 0);
+                       cv = getcvec(v, UCHAR_MAX, 0);
                        if (cv)
                        {
                                for (i = 0; i <= UCHAR_MAX; i++)
@@ -802,7 +770,7 @@ allcases(struct vars * v,           /* context */
        lc = pg_wc_tolower((chr) c);
        uc = pg_wc_toupper((chr) c);
 
-       cv = getcvec(v, 2, 0, 0);
+       cv = getcvec(v, 2, 0);
        addchr(cv, lc);
        if (lc != uc)
                addchr(cv, uc);
index ea382df7f91152671fb642b8e80dd8ce40f1ceb1..4e307f5c15224726085c62aa32023a0df015d138 100644 (file)
@@ -28,7 +28,7 @@
  * OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF
  * ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  *
- * $PostgreSQL: pgsql/src/backend/regex/regc_nfa.c,v 1.5 2008/01/03 20:47:55 tgl Exp $
+ * $PostgreSQL: pgsql/src/backend/regex/regc_nfa.c,v 1.6 2008/02/14 17:33:37 tgl Exp $
  *
  *
  * One or two things that technically ought to be in here
@@ -349,8 +349,6 @@ newarc(struct nfa * nfa,
 
        if (COLORED(a) && nfa->parent == NULL)
                colorchain(nfa->cm, a);
-
-       return;
 }
 
 /*
@@ -361,8 +359,6 @@ allocarc(struct nfa * nfa,
                 struct state * s)
 {
        struct arc *a;
-       struct arcbatch *new;
-       int                     i;
 
        /* shortcut */
        if (s->free == NULL && s->noas < ABSIZE)
@@ -375,22 +371,25 @@ allocarc(struct nfa * nfa,
        /* if none at hand, get more */
        if (s->free == NULL)
        {
-               new = (struct arcbatch *) MALLOC(sizeof(struct arcbatch));
-               if (new == NULL)
+               struct arcbatch *newAb;
+               int                     i;
+
+               newAb = (struct arcbatch *) MALLOC(sizeof(struct arcbatch));
+               if (newAb == NULL)
                {
                        NERR(REG_ESPACE);
                        return NULL;
                }
-               new->next = s->oas.next;
-               s->oas.next = new;
+               newAb->next = s->oas.next;
+               s->oas.next = newAb;
 
                for (i = 0; i < ABSIZE; i++)
                {
-                       new->a[i].type = 0;
-                       new->a[i].freechain = &new->a[i + 1];
+                       newAb->a[i].type = 0;
+                       newAb->a[i].freechain = &newAb->a[i + 1];
                }
-               new->a[ABSIZE - 1].freechain = NULL;
-               s->free = &new->a[0];
+               newAb->a[ABSIZE - 1].freechain = NULL;
+               s->free = &newAb->a[0];
        }
        assert(s->free != NULL);
 
@@ -495,20 +494,20 @@ cparc(struct nfa * nfa,
  */
 static void
 moveins(struct nfa * nfa,
-               struct state * old,
-               struct state * new)
+               struct state * oldState,
+               struct state * newState)
 {
        struct arc *a;
 
-       assert(old != new);
+       assert(oldState != newState);
 
-       while ((a = old->ins) != NULL)
+       while ((a = oldState->ins) != NULL)
        {
-               cparc(nfa, a, a->from, new);
+               cparc(nfa, a, a->from, newState);
                freearc(nfa, a);
        }
-       assert(old->nins == 0);
-       assert(old->ins == NULL);
+       assert(oldState->nins == 0);
+       assert(oldState->ins == NULL);
 }
 
 /*
@@ -516,15 +515,15 @@ moveins(struct nfa * nfa,
  */
 static void
 copyins(struct nfa * nfa,
-               struct state * old,
-               struct state * new)
+               struct state * oldState,
+               struct state * newState)
 {
        struct arc *a;
 
-       assert(old != new);
+       assert(oldState != newState);
 
-       for (a = old->ins; a != NULL; a = a->inchain)
-               cparc(nfa, a, a->from, new);
+       for (a = oldState->ins; a != NULL; a = a->inchain)
+               cparc(nfa, a, a->from, newState);
 }
 
 /*
@@ -532,16 +531,16 @@ copyins(struct nfa * nfa,
  */
 static void
 moveouts(struct nfa * nfa,
-                struct state * old,
-                struct state * new)
+                struct state * oldState,
+                struct state * newState)
 {
        struct arc *a;
 
-       assert(old != new);
+       assert(oldState != newState);
 
-       while ((a = old->outs) != NULL)
+       while ((a = oldState->outs) != NULL)
        {
-               cparc(nfa, a, new, a->to);
+               cparc(nfa, a, newState, a->to);
                freearc(nfa, a);
        }
 }
@@ -551,15 +550,15 @@ moveouts(struct nfa * nfa,
  */
 static void
 copyouts(struct nfa * nfa,
-                struct state * old,
-                struct state * new)
+                struct state * oldState,
+                struct state * newState)
 {
        struct arc *a;
 
-       assert(old != new);
+       assert(oldState != newState);
 
-       for (a = old->outs; a != NULL; a = a->outchain)
-               cparc(nfa, a, new, a->to);
+       for (a = oldState->outs; a != NULL; a = a->outchain)
+               cparc(nfa, a, newState, a->to);
 }
 
 /*
index 1eaca67b9a817bb4531be2ef30d1a1bcda9dac98..89f4f6f737ecb007f8f97fbc6b84199cf0fc4baf 100644 (file)
@@ -28,7 +28,7 @@
  * OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF
  * ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  *
- * $PostgreSQL: pgsql/src/backend/regex/regcomp.c,v 1.45 2007/10/06 16:05:54 tgl Exp $
+ * $PostgreSQL: pgsql/src/backend/regex/regcomp.c,v 1.46 2008/02/14 17:33:37 tgl Exp $
  *
  */
 
@@ -51,11 +51,9 @@ static void repeat(struct vars *, struct state *, struct state *, int, int);
 static void bracket(struct vars *, struct state *, struct state *);
 static void cbracket(struct vars *, struct state *, struct state *);
 static void brackpart(struct vars *, struct state *, struct state *);
-static chr *scanplain(struct vars *);
-static void leaders(struct vars *, struct cvec *);
+static const chr *scanplain(struct vars *);
 static void onechr(struct vars *, chr, struct state *, struct state *);
 static void dovec(struct vars *, struct cvec *, struct state *, struct state *);
-static celt nextleader(struct vars *, chr, chr);
 static void wordchrs(struct vars *);
 static struct subre *subre(struct vars *, int, int, struct state *, struct state *);
 static void freesubre(struct vars *, struct subre *);
@@ -74,12 +72,12 @@ static void rfree(regex_t *);
 static void dump(regex_t *, FILE *);
 static void dumpst(struct subre *, FILE *, int);
 static void stdump(struct subre *, FILE *, int);
-static char *stid(struct subre *, char *, size_t);
+static const char *stid(struct subre *, char *, size_t);
 #endif
 /* === regc_lex.c === */
 static void lexstart(struct vars *);
 static void prefixes(struct vars *);
-static void lexnest(struct vars *, chr *, chr *);
+static void lexnest(struct vars *, const chr *, const chr *);
 static void lexword(struct vars *);
 static int     next(struct vars *);
 static int     lexescape(struct vars *);
@@ -87,7 +85,7 @@ static chr    lexdigits(struct vars *, int, int, int);
 static int     brenext(struct vars *, chr);
 static void skip(struct vars *);
 static chr     newline(void);
-static chr     chrnamed(struct vars *, chr *, chr *, chr);
+static chr     chrnamed(struct vars *, const chr *, const chr *, chr);
 
 /* === regc_color.c === */
 static void initcm(struct vars *, struct colormap *);
@@ -105,7 +103,6 @@ static void subblock(struct vars *, chr, struct state *, struct state *);
 static void okcolors(struct nfa *, struct colormap *);
 static void colorchain(struct colormap *, struct arc *);
 static void uncolorchain(struct colormap *, struct arc *);
-static int     singleton(struct colormap *, chr c);
 static void rainbow(struct nfa *, struct colormap *, int, pcolor, struct state *, struct state *);
 static void colorcomplement(struct nfa *, struct colormap *, int, struct state *, struct state *, struct state *);
 
@@ -168,13 +165,11 @@ static void dumpcnfa(struct cnfa *, FILE *);
 static void dumpcstate(int, struct carc *, struct cnfa *, FILE *);
 #endif
 /* === regc_cvec.c === */
-static struct cvec *newcvec(int, int, int);
+static struct cvec *newcvec(int, int);
 static struct cvec *clearcvec(struct cvec *);
 static void addchr(struct cvec *, chr);
 static void addrange(struct cvec *, chr, chr);
-static void addmcce(struct cvec *, chr *, chr *);
-static int     haschr(struct cvec *, chr);
-static struct cvec *getcvec(struct vars *, int, int, int);
+static struct cvec *getcvec(struct vars *, int, int);
 static void freecvec(struct cvec *);
 
 /* === regc_locale.c === */
@@ -189,14 +184,11 @@ static int        pg_wc_ispunct(pg_wchar c);
 static int     pg_wc_isspace(pg_wchar c);
 static pg_wchar pg_wc_toupper(pg_wchar c);
 static pg_wchar pg_wc_tolower(pg_wchar c);
-static int     nmcces(struct vars *);
-static int     nleaders(struct vars *);
-static struct cvec *allmcces(struct vars *, struct cvec *);
-static celt element(struct vars *, chr *, chr *);
+static celt element(struct vars *, const chr *, const chr *);
 static struct cvec *range(struct vars *, celt, celt, int);
 static int     before(celt, celt);
 static struct cvec *eclass(struct vars *, celt, int);
-static struct cvec *cclass(struct vars *, chr *, chr *, int);
+static struct cvec *cclass(struct vars *, const chr *, const chr *, int);
 static struct cvec *allcases(struct vars *, chr);
 static int     cmp(const chr *, const chr *, size_t);
 static int     casecmp(const chr *, const chr *, size_t);
@@ -206,10 +198,10 @@ static int        casecmp(const chr *, const chr *, size_t);
 struct vars
 {
        regex_t    *re;
-       chr                *now;                        /* scan pointer into string */
-       chr                *stop;                       /* end of string */
-       chr                *savenow;            /* saved now and stop for "subroutine call" */
-       chr                *savestop;
+       const chr  *now;                        /* scan pointer into string */
+       const chr  *stop;                       /* end of string */
+       const chr  *savenow;            /* saved now and stop for "subroutine call" */
+       const chr  *savestop;
        int                     err;                    /* error code (0 if none) */
        int                     cflags;                 /* copy of compile flags */
        int                     lasttype;               /* type of previous token */
@@ -230,10 +222,6 @@ struct vars
        int                     ntree;                  /* number of tree nodes */
        struct cvec *cv;                        /* interface cvec */
        struct cvec *cv2;                       /* utility cvec */
-       struct cvec *mcces;                     /* collating-element information */
-#define  ISCELEADER(v,c) ((v)->mcces != NULL && haschr((v)->mcces, (c)))
-       struct state *mccepbegin;       /* in nfa, start of MCCE prototypes */
-       struct state *mccepend;         /* in nfa, end of MCCE prototypes */
        struct subre *lacons;           /* lookahead-constraint vector */
        int                     nlacons;                /* size of lacons */
 };
@@ -275,9 +263,8 @@ struct vars
 #define PREFER 'P'                             /* length preference */
 
 /* is an arc colored, and hence on a color chain? */
-#define COLORED(a)     ((a)->type == PLAIN || (a)->type == AHEAD || \
-                                                       (a)->type == BEHIND)
-
+#define COLORED(a) \
+       ((a)->type == PLAIN || (a)->type == AHEAD || (a)->type == BEHIND)
 
 
 /* static function list */
@@ -322,7 +309,7 @@ pg_regcomp(regex_t *re,
 
        /* initial setup (after which freev() is callable) */
        v->re = re;
-       v->now = (chr *) string;
+       v->now = string;
        v->stop = v->now + len;
        v->savenow = v->savestop = NULL;
        v->err = 0;
@@ -341,7 +328,6 @@ pg_regcomp(regex_t *re,
        v->treefree = NULL;
        v->cv = NULL;
        v->cv2 = NULL;
-       v->mcces = NULL;
        v->lacons = NULL;
        v->nlacons = 0;
        re->re_magic = REMAGIC;
@@ -363,19 +349,9 @@ pg_regcomp(regex_t *re,
        ZAPCNFA(g->search);
        v->nfa = newnfa(v, v->cm, (struct nfa *) NULL);
        CNOERR();
-       v->cv = newcvec(100, 20, 10);
+       v->cv = newcvec(100, 20);
        if (v->cv == NULL)
                return freev(v, REG_ESPACE);
-       i = nmcces(v);
-       if (i > 0)
-       {
-               v->mcces = newcvec(nleaders(v), 0, i);
-               CNOERR();
-               v->mcces = allmcces(v, v->mcces);
-               leaders(v, v->mcces);
-               addmcce(v->mcces, (chr *) NULL, (chr *) NULL);  /* dummy */
-       }
-       CNOERR();
 
        /* parsing */
        lexstart(v);                            /* also handles prefixes */
@@ -525,8 +501,6 @@ freev(struct vars * v,
                freecvec(v->cv);
        if (v->cv2 != NULL)
                freecvec(v->cv2);
-       if (v->mcces != NULL)
-               freecvec(v->mcces);
        if (v->lacons != NULL)
                freelacons(v->lacons, v->nlacons);
        ERR(err);                                       /* nop if err==0 */
@@ -583,15 +557,14 @@ makesearch(struct vars * v,
                for (b = s->ins; b != NULL; b = b->inchain)
                        if (b->from != pre)
                                break;
-               if (b != NULL)
-               {                                               /* must be split */
-                       if (s->tmp == NULL)
-                       {                                       /* if not already in the list */
-                               /* (fixes bugs 505048, 230589, */
-                               /* 840258, 504785) */
-                               s->tmp = slist;
-                               slist = s;
-                       }
+               if (b != NULL && s->tmp == NULL)
+               {
+                       /*
+                        * Must be split if not already in the list (fixes bugs 505048,
+                        * 230589, 840258, 504785).
+                        */
+                       s->tmp = slist;
+                       slist = s;
                }
        }
 
@@ -1338,13 +1311,6 @@ cbracket(struct vars * v,
 {
        struct state *left = newstate(v->nfa);
        struct state *right = newstate(v->nfa);
-       struct state *s;
-       struct arc *a;                          /* arc from lp */
-       struct arc *ba;                         /* arc from left, from bracket() */
-       struct arc *pa;                         /* MCCE-prototype arc */
-       color           co;
-       chr                *p;
-       int                     i;
 
        NOERR();
        bracket(v, left, right);
@@ -1354,65 +1320,13 @@ cbracket(struct vars * v,
 
        assert(lp->nouts == 0);         /* all outarcs will be ours */
 
-       /* easy part of complementing */
+       /*
+        * Easy part of complementing, and all there is to do since the MCCE code
+        * was removed.
+        */
        colorcomplement(v->nfa, v->cm, PLAIN, left, lp, rp);
        NOERR();
-       if (v->mcces == NULL)
-       {                                                       /* no MCCEs -- we're done */
-               dropstate(v->nfa, left);
-               assert(right->nins == 0);
-               freestate(v->nfa, right);
-               return;
-       }
-
-       /* but complementing gets messy in the presence of MCCEs... */
-       NOTE(REG_ULOCALE);
-       for (p = v->mcces->chrs, i = v->mcces->nchrs; i > 0; p++, i--)
-       {
-               co = GETCOLOR(v->cm, *p);
-               a = findarc(lp, PLAIN, co);
-               ba = findarc(left, PLAIN, co);
-               if (ba == NULL)
-               {
-                       assert(a != NULL);
-                       freearc(v->nfa, a);
-               }
-               else
-                       assert(a == NULL);
-               s = newstate(v->nfa);
-               NOERR();
-               newarc(v->nfa, PLAIN, co, lp, s);
-               NOERR();
-               pa = findarc(v->mccepbegin, PLAIN, co);
-               assert(pa != NULL);
-               if (ba == NULL)
-               {                                               /* easy case, need all of them */
-                       cloneouts(v->nfa, pa->to, s, rp, PLAIN);
-                       newarc(v->nfa, '$', 1, s, rp);
-                       newarc(v->nfa, '$', 0, s, rp);
-                       colorcomplement(v->nfa, v->cm, AHEAD, pa->to, s, rp);
-               }
-               else
-               {                                               /* must be selective */
-                       if (findarc(ba->to, '$', 1) == NULL)
-                       {
-                               newarc(v->nfa, '$', 1, s, rp);
-                               newarc(v->nfa, '$', 0, s, rp);
-                               colorcomplement(v->nfa, v->cm, AHEAD, pa->to,
-                                                               s, rp);
-                       }
-                       for (pa = pa->to->outs; pa != NULL; pa = pa->outchain)
-                               if (findarc(ba->to, PLAIN, pa->co) == NULL)
-                                       newarc(v->nfa, PLAIN, pa->co, s, rp);
-                       if (s->nouts == 0)      /* limit of selectivity: none */
-                               dropstate(v->nfa, s);   /* frees arc too */
-               }
-               NOERR();
-       }
-
-       delsub(v->nfa, left, right);
-       assert(left->nouts == 0);
-       freestate(v->nfa, left);
+       dropstate(v->nfa, left);
        assert(right->nins == 0);
        freestate(v->nfa, right);
 }
@@ -1428,8 +1342,8 @@ brackpart(struct vars * v,
        celt            startc;
        celt            endc;
        struct cvec *cv;
-       chr                *startp;
-       chr                *endp;
+       const chr  *startp;
+       const chr  *endp;
        chr                     c[1];
 
        /* parse something, get rid of special cases, take shortcuts */
@@ -1442,8 +1356,8 @@ brackpart(struct vars * v,
                case PLAIN:
                        c[0] = v->nextvalue;
                        NEXT();
-                       /* shortcut for ordinary chr (not range, not MCCE leader) */
-                       if (!SEE(RANGE) && !ISCELEADER(v, c[0]))
+                       /* shortcut for ordinary chr (not range) */
+                       if (!SEE(RANGE))
                        {
                                onechr(v, c[0], lp, rp);
                                return;
@@ -1533,10 +1447,10 @@ brackpart(struct vars * v,
  * Certain bits of trickery in lex.c know that this code does not try
  * to look past the final bracket of the [. etc.
  */
-static chr *                                   /* just after end of sequence */
+static const chr *                             /* just after end of sequence */
 scanplain(struct vars * v)
 {
-       chr                *endp;
+       const chr  *endp;
 
        assert(SEE(COLLEL) || SEE(ECLASS) || SEE(CCLASS));
        NEXT();
@@ -1554,52 +1468,6 @@ scanplain(struct vars * v)
        return endp;
 }
 
-/*
- * leaders - process a cvec of collating elements to also include leaders
- * Also gives all characters involved their own colors, which is almost
- * certainly necessary, and sets up little disconnected subNFA.
- */
-static void
-leaders(struct vars * v,
-               struct cvec * cv)
-{
-       int                     mcce;
-       chr                *p;
-       chr                     leader;
-       struct state *s;
-       struct arc *a;
-
-       v->mccepbegin = newstate(v->nfa);
-       v->mccepend = newstate(v->nfa);
-       NOERR();
-
-       for (mcce = 0; mcce < cv->nmcces; mcce++)
-       {
-               p = cv->mcces[mcce];
-               leader = *p;
-               if (!haschr(cv, leader))
-               {
-                       addchr(cv, leader);
-                       s = newstate(v->nfa);
-                       newarc(v->nfa, PLAIN, subcolor(v->cm, leader),
-                                  v->mccepbegin, s);
-                       okcolors(v->nfa, v->cm);
-               }
-               else
-               {
-                       a = findarc(v->mccepbegin, PLAIN,
-                                               GETCOLOR(v->cm, leader));
-                       assert(a != NULL);
-                       s = a->to;
-                       assert(s != v->mccepend);
-               }
-               p++;
-               assert(*p != 0 && *(p + 1) == 0);               /* only 2-char MCCEs for now */
-               newarc(v->nfa, PLAIN, subcolor(v->cm, *p), s, v->mccepend);
-               okcolors(v->nfa, v->cm);
-       }
-}
-
 /*
  * onechr - fill in arcs for a plain character, and possible case complements
  * This is mostly a shortcut for efficient handling of the common case.
@@ -1622,7 +1490,6 @@ onechr(struct vars * v,
 
 /*
  * dovec - fill in arcs for each element of a cvec
- * This one has to handle the messy cases, like MCCEs and MCCE leaders.
  */
 static void
 dovec(struct vars * v,
@@ -1633,47 +1500,14 @@ dovec(struct vars * v,
        chr                     ch,
                                from,
                                to;
-       celt            ce;
-       chr                *p;
+       const chr  *p;
        int                     i;
-       color           co;
-       struct cvec *leads;
-       struct arc *a;
-       struct arc *pa;                         /* arc in prototype */
-       struct state *s;
-       struct state *ps;                       /* state in prototype */
-
-       /* need a place to store leaders, if any */
-       if (nmcces(v) > 0)
-       {
-               assert(v->mcces != NULL);
-               if (v->cv2 == NULL || v->cv2->nchrs < v->mcces->nchrs)
-               {
-                       if (v->cv2 != NULL)
-                               free(v->cv2);
-                       v->cv2 = newcvec(v->mcces->nchrs, 0, v->mcces->nmcces);
-                       NOERR();
-                       leads = v->cv2;
-               }
-               else
-                       leads = clearcvec(v->cv2);
-       }
-       else
-               leads = NULL;
 
-       /* first, get the ordinary characters out of the way */
+       /* ordinary characters */
        for (p = cv->chrs, i = cv->nchrs; i > 0; p++, i--)
        {
                ch = *p;
-               if (!ISCELEADER(v, ch))
-                       newarc(v->nfa, PLAIN, subcolor(v->cm, ch), lp, rp);
-               else
-               {
-                       assert(singleton(v->cm, ch));
-                       assert(leads != NULL);
-                       if (!haschr(leads, ch))
-                               addchr(leads, ch);
-               }
+               newarc(v->nfa, PLAIN, subcolor(v->cm, ch), lp, rp);
        }
 
        /* and the ranges */
@@ -1681,103 +1515,9 @@ dovec(struct vars * v,
        {
                from = *p;
                to = *(p + 1);
-               while (from <= to && (ce = nextleader(v, from, to)) != NOCELT)
-               {
-                       if (from < ce)
-                               subrange(v, from, ce - 1, lp, rp);
-                       assert(singleton(v->cm, ce));
-                       assert(leads != NULL);
-                       if (!haschr(leads, ce))
-                               addchr(leads, ce);
-                       from = ce + 1;
-               }
                if (from <= to)
                        subrange(v, from, to, lp, rp);
        }
-
-       if ((leads == NULL || leads->nchrs == 0) && cv->nmcces == 0)
-               return;
-
-       /* deal with the MCCE leaders */
-       NOTE(REG_ULOCALE);
-       for (p = leads->chrs, i = leads->nchrs; i > 0; p++, i--)
-       {
-               co = GETCOLOR(v->cm, *p);
-               a = findarc(lp, PLAIN, co);
-               if (a != NULL)
-                       s = a->to;
-               else
-               {
-                       s = newstate(v->nfa);
-                       NOERR();
-                       newarc(v->nfa, PLAIN, co, lp, s);
-                       NOERR();
-               }
-               pa = findarc(v->mccepbegin, PLAIN, co);
-               assert(pa != NULL);
-               ps = pa->to;
-               newarc(v->nfa, '$', 1, s, rp);
-               newarc(v->nfa, '$', 0, s, rp);
-               colorcomplement(v->nfa, v->cm, AHEAD, ps, s, rp);
-               NOERR();
-       }
-
-       /* and the MCCEs */
-       for (i = 0; i < cv->nmcces; i++)
-       {
-               p = cv->mcces[i];
-               assert(singleton(v->cm, *p));
-               if (!singleton(v->cm, *p))
-               {
-                       ERR(REG_ASSERT);
-                       return;
-               }
-               ch = *p++;
-               co = GETCOLOR(v->cm, ch);
-               a = findarc(lp, PLAIN, co);
-               if (a != NULL)
-                       s = a->to;
-               else
-               {
-                       s = newstate(v->nfa);
-                       NOERR();
-                       newarc(v->nfa, PLAIN, co, lp, s);
-                       NOERR();
-               }
-               assert(*p != 0);                /* at least two chars */
-               assert(singleton(v->cm, *p));
-               ch = *p++;
-               co = GETCOLOR(v->cm, ch);
-               assert(*p == 0);                /* and only two, for now */
-               newarc(v->nfa, PLAIN, co, s, rp);
-               NOERR();
-       }
-}
-
-/*
- * nextleader - find next MCCE leader within range
- */
-static celt                                            /* NOCELT means none */
-nextleader(struct vars * v,
-                  chr from,
-                  chr to)
-{
-       int                     i;
-       chr                *p;
-       chr                     ch;
-       celt            it = NOCELT;
-
-       if (v->mcces == NULL)
-               return it;
-
-       for (i = v->mcces->nchrs, p = v->mcces->chrs; i > 0; i--, p++)
-       {
-               ch = *p;
-               if (from <= ch && ch <= to)
-                       if (it == NOCELT || ch < it)
-                               it = ch;
-       }
-       return it;
 }
 
 /*
@@ -1825,9 +1565,8 @@ subre(struct vars * v,
          struct state * begin,
          struct state * end)
 {
-       struct subre *ret;
+       struct subre *ret = v->treefree;
 
-       ret = v->treefree;
        if (ret != NULL)
                v->treefree = ret->left;
        else
@@ -1906,14 +1645,13 @@ static void
 optst(struct vars * v,
          struct subre * t)
 {
-       if (t == NULL)
-               return;
-
-       /* recurse through children */
-       if (t->left != NULL)
-               optst(v, t->left);
-       if (t->right != NULL)
-               optst(v, t->right);
+       /*
+        * DGP (2007-11-13): I assume it was the programmer's intent to eventually
+        * come back and add code to optimize subRE trees, but the routine coded
+        * just spends effort traversing the tree and doing nothing. We can do
+        * nothing with less effort.
+        */
+       return;
 }
 
 /*
@@ -2207,8 +1945,8 @@ stdump(struct subre * t,
        {
                fprintf(f, "\n");
                dumpcnfa(&t->cnfa, f);
-               fprintf(f, "\n");
        }
+       fprintf(f, "\n");
        if (t->left != NULL)
                stdump(t->left, f, nfapresent);
        if (t->right != NULL)
@@ -2218,7 +1956,7 @@ stdump(struct subre * t,
 /*
  * stid - identify a subtree node for dumping
  */
-static char *                                  /* points to buf or constant string */
+static const char *                            /* points to buf or constant string */
 stid(struct subre * t,
         char *buf,
         size_t bufsize)
index 3b20446350770445c5c4ff49946c5e463bef88d4..dfcb462e0115aa67b089248822d2cb4a5c206969 100644 (file)
@@ -27,7 +27,7 @@
  * OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF
  * ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  *
- * $PostgreSQL: pgsql/src/backend/regex/regerror.c,v 1.27 2003/11/29 19:51:55 pgsql Exp $
+ * $PostgreSQL: pgsql/src/backend/regex/regerror.c,v 1.28 2008/02/14 17:33:37 tgl Exp $
  *
  */
 
@@ -40,8 +40,8 @@ static char unk[] = "*** unknown regex error code 0x%x ***";
 static struct rerr
 {
        int                     code;
-       char       *name;
-       char       *explain;
+       const char *name;
+       const char *explain;
 }      rerrs[] =
 
 {
@@ -63,7 +63,7 @@ pg_regerror(int errcode,              /* error code, or REG_ATOI or REG_ITOA */
                        size_t errbuf_size) /* available space in errbuf, can be 0 */
 {
        struct rerr *r;
-       char       *msg;
+       const char *msg;
        char            convbuf[sizeof(unk) + 50];              /* 50 = plenty for int */
        size_t          len;
        int                     icode;
index 8471fb0b1cc00b439ac1add59b09990ffe67ac79..269f926be856bdaa8f4fce235547d43d7705233f 100644 (file)
@@ -25,7 +25,7 @@
  * OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF
  * ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  *
- * $PostgreSQL: pgsql/src/include/regex/regcustom.h,v 1.6 2007/10/06 16:01:51 tgl Exp $
+ * $PostgreSQL: pgsql/src/include/regex/regcustom.h,v 1.7 2008/02/14 17:33:37 tgl Exp $
  */
 
 /* headers if any */
@@ -47,9 +47,9 @@
 /* internal character type and related */
 typedef pg_wchar chr;                  /* the type itself */
 typedef unsigned uchr;                 /* unsigned type that will hold a chr */
-typedef int celt;                              /* type to hold chr, MCCE number, or NOCELT */
+typedef int celt;                              /* type to hold chr, or NOCELT */
 
-#define NOCELT (-1)                    /* celt value which is not valid chr or MCCE */
+#define NOCELT (-1)                    /* celt value which is not valid chr */
 #define CHR(c) ((unsigned char) (c))   /* turn char literal into chr literal */
 #define DIGITVAL(c) ((c)-'0')  /* turn chr digit into its value */
 #define CHRBITS 32                             /* bits in a chr; must not use sizeof */
index 327808338a004a7d05261a2ad0a7a0b68a3b2d28..52f2157535a13c9a12b99a95f2f7d7dc7ae1c0a8 100644 (file)
@@ -27,7 +27,7 @@
  * OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF
  * ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  *
- * $PostgreSQL: pgsql/src/include/regex/regguts.h,v 1.6 2008/01/03 20:47:55 tgl Exp $
+ * $PostgreSQL: pgsql/src/include/regex/regguts.h,v 1.7 2008/02/14 17:33:37 tgl Exp $
  */
 
 
@@ -181,7 +181,7 @@ union tree
 #define tcolor colors.ccolor
 #define tptr   ptrs.pptr
 
-/* internal per-color structure for the color machinery */
+/* internal per-color descriptor structure for the color machinery */
 struct colordesc
 {
        uchr            nchrs;                  /* number of chars of this color */
@@ -228,11 +228,11 @@ struct colormap
 #endif
 
 
-
 /*
  * Interface definitions for locale-interface functions in locale.c.
- * Multi-character collating elements (MCCEs) cause most of the trouble.
  */
+
+/* Representation of a set of characters. */
 struct cvec
 {
        int                     nchrs;                  /* number of chrs */
@@ -241,17 +241,9 @@ struct cvec
        int                     nranges;                /* number of ranges (chr pairs) */
        int                     rangespace;             /* number of chrs possible */
        chr                *ranges;                     /* pointer to vector of chr pairs */
-       int                     nmcces;                 /* number of MCCEs */
-       int                     mccespace;              /* number of MCCEs possible */
-       int                     nmccechrs;              /* number of chrs used for MCCEs */
-       chr                *mcces[1];           /* pointers to 0-terminated MCCEs */
-       /* and both batches of chrs are on the end */
+       /* both batches of chrs are on the end */
 };
 
-/* caution:  this value cannot be changed easily */
-#define MAXMCCE 2                              /* length of longest MCCE */
-
-
 
 /*
  * definitions for NFA internal representation