]> git.cameronkatri.com Git - mandoc.git/commitdiff
Use strcspn() in term_strlen(). Clarifies the code.
authorKristaps Dzonsons <kristaps@bsd.lv>
Sun, 15 May 2011 14:50:01 +0000 (14:50 +0000)
committerKristaps Dzonsons <kristaps@bsd.lv>
Sun, 15 May 2011 14:50:01 +0000 (14:50 +0000)
term.c

diff --git a/term.c b/term.c
index 92a673770bd237e447dee781507a6f3355c87f8d..1c5e8ff0f0869d8cf86d770aa766bdb80a99a199 100644 (file)
--- a/term.c
+++ b/term.c
@@ -1,4 +1,4 @@
-/*     $Id: term.c,v 1.188 2011/05/14 18:15:20 kristaps Exp $ */
+/*     $Id: term.c,v 1.189 2011/05/15 14:50:01 kristaps Exp $ */
 /*
  * Copyright (c) 2008, 2009, 2010 Kristaps Dzonsons <kristaps@bsd.lv>
  * Copyright (c) 2010, 2011 Ingo Schwarze <schwarze@openbsd.org>
@@ -606,8 +606,8 @@ term_strlen(const struct termp *p, const char *cp)
 {
        size_t           sz, rsz, i;
        int              ssz;
-       enum mandoc_esc  esc;
        const char      *seq, *rhs;
+       static const char rej[] = { '\\', ASCII_HYPH, ASCII_NBRSP, '\0' };
 
        /*
         * Account for escaped sequences within string length
@@ -616,15 +616,17 @@ term_strlen(const struct termp *p, const char *cp)
         */
 
        sz = 0;
-       while ('\0' != *cp)
+       while ('\0' != *cp) {
+               rsz = strcspn(cp, rej);
+               for (i = 0; i < rsz; i++)
+                       sz += (*p->width)(p, *cp++);
+
                switch (*cp) {
                case ('\\'):
-                       ++cp;
-                       esc = mandoc_escape(&cp, &seq, &ssz);
-                       if (ESCAPE_ERROR == esc)
+                       cp++;
+                       switch (mandoc_escape(&cp, &seq, &ssz)) {
+                       case (ESCAPE_ERROR):
                                return(sz);
-
-                       switch (esc) {
                        case (ESCAPE_PREDEF):
                                rhs = mchars_res2str
                                        (p->symtab, seq, ssz, &rsz);
@@ -659,14 +661,13 @@ term_strlen(const struct termp *p, const char *cp)
                        cp++;
                        break;
                default:
-                       sz += (*p->width)(p, *cp++);
                        break;
                }
+       }
 
        return(sz);
 }
 
-
 /* ARGSUSED */
 size_t
 term_vspan(const struct termp *p, const struct roffsu *su)
@@ -703,7 +704,6 @@ term_vspan(const struct termp *p, const struct roffsu *su)
                        r);
 }
 
-
 size_t
 term_hspan(const struct termp *p, const struct roffsu *su)
 {