diff options
author | ache <ache@FreeBSD.org> | 2003-08-04 05:22:06 +0000 |
---|---|---|
committer | ache <ache@FreeBSD.org> | 2003-08-04 05:22:06 +0000 |
commit | 71cdd2eb3d19206d1112c7b8d3298a79e8ed670f (patch) | |
tree | 26e2b652d05c032fda88b5b3b335a465f0a76056 /usr.bin/tr/str.c | |
parent | d5a33e59d13687ecea3a42a2b0b563e2b45dfaa2 (diff) | |
download | FreeBSD-src-71cdd2eb3d19206d1112c7b8d3298a79e8ed670f.zip FreeBSD-src-71cdd2eb3d19206d1112c7b8d3298a79e8ed670f.tar.gz |
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."
Diffstat (limited to 'usr.bin/tr/str.c')
-rw-r--r-- | usr.bin/tr/str.c | 28 |
1 files changed, 17 insertions, 11 deletions
diff --git a/usr.bin/tr/str.c b/usr.bin/tr/str.c index a307bb8..9461997 100644 --- a/usr.bin/tr/str.c +++ b/usr.bin/tr/str.c @@ -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); } |