android: regex: revert fixes
authorFerry Huberts <ferry.huberts@pelagic.nl>
Wed, 20 Jun 2012 18:57:03 +0000 (20:57 +0200)
committerFerry Huberts <ferry.huberts@pelagic.nl>
Wed, 20 Jun 2012 18:57:03 +0000 (20:57 +0200)
because Hans politely asked me to

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
android/regex/cclass.h
android/regex/cname.h
android/regex/engine.c
android/regex/regcomp.c
android/regex/regerror.c

index 1f78a64..d105491 100644 (file)
@@ -37,9 +37,9 @@
 
 /* character-class table */
 static const struct cclass {
-       const char *name;
-       const char *chars;
-       const char *multis;
+       char *name;
+       char *chars;
+       char *multis;
 } cclasses[] = {
        { "alnum",      "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz\
 0123456789",                           ""} ,
index c61956d..b674b68 100644 (file)
@@ -37,8 +37,8 @@
 
 /* character-name table */
 static const struct cname {
-       const char *name;
-       const char code;
+       char *name;
+       char code;
 } cnames[] = {
        { "NUL",                        '\0' },
        { "SOH",                        '\001' },
index e211b9a..eae6ff2 100644 (file)
@@ -318,8 +318,6 @@ dissect(struct match *m, char *start, char *stop, sopno startst, sopno stopst)
                        while (OP(m->g->strip[es]) != O_CH)
                                es += OPND(m->g->strip[es]);
                        break;
-               default:
-                       break;
                }
                es++;
 
@@ -699,12 +697,12 @@ fast(struct match *m, char *start, char *stop, sopno startst, sopno stopst)
                /* is there an EOL and/or BOL between lastc and c? */
                flagch = '\0';
                i = 0;
-               if ( (lastc == '\n' && (m->g->cflags&REG_NEWLINE)) ||
+               if ( (lastc == '\n' && m->g->cflags&REG_NEWLINE) ||
                                (lastc == OUT && !(m->eflags&REG_NOTBOL)) ) {
                        flagch = BOL;
                        i = m->g->nbol;
                }
-               if ( (c == '\n' && (m->g->cflags&REG_NEWLINE)) ||
+               if ( (c == '\n' && m->g->cflags&REG_NEWLINE) ||
                                (c == OUT && !(m->eflags&REG_NOTEOL)) ) {
                        flagch = (flagch == BOL) ? BOLEOL : EOL;
                        i += m->g->neol;
@@ -781,12 +779,12 @@ slow(struct match *m, char *start, char *stop, sopno startst, sopno stopst)
                /* is there an EOL and/or BOL between lastc and c? */
                flagch = '\0';
                i = 0;
-               if ( (lastc == '\n' && (m->g->cflags&REG_NEWLINE)) ||
+               if ( (lastc == '\n' && m->g->cflags&REG_NEWLINE) ||
                                (lastc == OUT && !(m->eflags&REG_NOTBOL)) ) {
                        flagch = BOL;
                        i = m->g->nbol;
                }
-               if ( (c == '\n' && (m->g->cflags&REG_NEWLINE)) ||
+               if ( (c == '\n' && m->g->cflags&REG_NEWLINE) ||
                                (c == OUT && !(m->eflags&REG_NOTEOL)) ) {
                        flagch = (flagch == BOL) ? BOLEOL : EOL;
                        i += m->g->neol;
index fe9d760..19f4790 100644 (file)
@@ -168,7 +168,7 @@ regcomp(regex_t *preg, const char *pattern, int cflags)
                        return(REG_INVARG);
                len = preg->re_endp - pattern;
        } else
-               len = strlen(pattern);
+               len = strlen((char *)pattern);
 
        /* do the mallocs early so failure handling is easy */
        g = (struct re_guts *)malloc(sizeof(struct re_guts) +
@@ -366,7 +366,7 @@ p_ere_exp(struct parse *p)
                break;
        case '{':               /* okay as ordinary except if digit follows */
                REQUIRE(!MORE() || !isdigit((uch)PEEK()), REG_BADRPT);
-               /* no break */
+               /* FALLTHROUGH */
        default:
                ordinary(p, c);
                break;
@@ -562,7 +562,7 @@ p_simp_re(struct parse *p,
                break;
        case '*':
                REQUIRE(starordinary, REG_BADRPT);
-               /* no break */
+               /* FALLTHROUGH */
        default:
                ordinary(p, (char)c);
                break;
@@ -1190,7 +1190,7 @@ mcadd( struct parse *p, cset *cs, char *cp)
  */
 /* ARGSUSED */
 static void
-mcinvert(struct parse *p __attribute__((unused)), cset *cs)
+mcinvert(struct parse *p, cset *cs)
 {
        assert(cs->multis == NULL);     /* xxx */
 }
@@ -1203,7 +1203,7 @@ mcinvert(struct parse *p __attribute__((unused)), cset *cs)
  */
 /* ARGSUSED */
 static void
-mccase(struct parse *p __attribute__((unused)), cset *cs)
+mccase(struct parse *p, cset *cs)
 {
        assert(cs->multis == NULL);     /* xxx */
 }
@@ -1451,7 +1451,7 @@ findmust(struct parse *p, struct re_guts *g)
                                        return;
                                }
                        } while (OP(s) != O_QUEST && OP(s) != O_CH);
-                       /* no break */
+                       /* fallthrough */
                default:                /* things that break a sequence */
                        if (newlen > g->mlen) {         /* ends one */
                                start = newstart;
@@ -1509,8 +1509,6 @@ pluscount(struct parse *p, struct re_guts *g)
                                maxnest = plusnest;
                        plusnest--;
                        break;
-               default:
-                       break;
                }
        } while (OP(s) != OEND);
        if (plusnest != 0)
index 5448f2b..838ec8f 100644 (file)
@@ -48,8 +48,8 @@ static char *regatoi(const regex_t *, char *, int);
 
 static const struct rerr {
        int code;
-       const char *name;
-       const char *explain;
+       char *name;
+       char *explain;
 } rerrs[] = {
        { REG_NOMATCH,  "REG_NOMATCH",  "regexec() failed to match" },
        { REG_BADPAT,   "REG_BADPAT",   "invalid regular expression" },