No functional changes, just code reorganization from prev. commit, it

makes one malloc unneeded, removes two bzero's and makes code more readable.

"Bright ideas comes only _after_ commits."
This commit is contained in:
Andrey A. Chernov 2003-08-04 05:22:06 +00:00
parent 46e31b2612
commit 30c1156451
Notes: svn2git 2020-12-20 02:59:44 +00:00
svn path=/head/; revision=118415
2 changed files with 18 additions and 12 deletions

View File

@ -40,7 +40,7 @@
typedef struct {
enum { STRING1, STRING2 } which;
enum { EOS, INFINITE, NORMAL, SEQUENCE,
enum { EOS, INFINITE, NORMAL, RANGE, SEQUENCE,
SET, SET_UPPER, SET_LOWER } state;
int cnt; /* character count */
int lastch; /* last character */

View File

@ -93,6 +93,13 @@ next(s)
if (s->str[0] == '-' && genrange(s, is_octal))
return (next(s));
return (1);
case RANGE:
if (s->cnt-- == 0) {
s->state = NORMAL;
return (next(s));
}
++s->lastch;
return (1);
case SEQUENCE:
if (s->cnt-- == 0) {
s->state = NORMAL;
@ -200,7 +207,6 @@ genclass(s)
if ((cp->set = p = malloc((NCHARS + 1) * sizeof(int))) == NULL)
err(1, "genclass() malloc");
bzero(p, (NCHARS + 1) * sizeof(int));
for (cnt = 0, func = cp->func; cnt < NCHARS; ++cnt)
if ((func)(cnt))
*p++ = cnt;
@ -290,24 +296,24 @@ genrange(STR *s, int was_octal)
s->str = savestart;
return (0);
}
if (octal) {
s->cnt = stopval - s->lastch + 1;
s->state = RANGE;
--s->lastch;
return (1);
}
if ((s->set = p = malloc((NCHARS + 1) * sizeof(int))) == NULL)
err(1, "genrange() malloc");
bzero(p, (NCHARS + 1) * sizeof(int));
if (octal) {
for (cnt = s->lastch; cnt <= stopval; cnt++)
for (cnt = 0; cnt < NCHARS; cnt++)
if (charcoll((const void *)&cnt, (const void *)&(s->lastch)) >= 0 &&
charcoll((const void *)&cnt, (const void *)&stopval) <= 0)
*p++ = cnt;
} else {
for (cnt = 0; cnt < NCHARS; cnt++)
if (charcoll((const void *)&cnt, (const void *)&(s->lastch)) >= 0 &&
charcoll((const void *)&cnt, (const void *)&stopval) <= 0)
*p++ = cnt;
}
*p = OOBCH;
n = p - s->set;
s->cnt = 0;
s->state = SET;
if (!octal && n > 1)
if (n > 1)
mergesort(s->set, n, sizeof(*(s->set)), charcoll);
return (1);
}