]> git.cameronkatri.com Git - mandoc.git/blobdiff - mandoc.c
fix -Tman font handling for:
[mandoc.git] / mandoc.c
index 20a6abf10dce717851011601316421419405e210..420e8aa5443bcf7cd7117534426b099dd0bffc45 100644 (file)
--- a/mandoc.c
+++ b/mandoc.c
@@ -1,4 +1,4 @@
-/*     $Id: mandoc.c,v 1.64 2012/05/31 22:34:06 schwarze Exp $ */
+/*     $Id: mandoc.c,v 1.66 2012/06/12 20:21:04 kristaps Exp $ */
 /*
  * Copyright (c) 2008, 2009, 2010, 2011 Kristaps Dzonsons <kristaps@bsd.lv>
  * Copyright (c) 2011, 2012 Ingo Schwarze <schwarze@openbsd.org>
@@ -42,20 +42,33 @@ static      char    *time2a(time_t);
 enum mandoc_esc
 mandoc_escape(const char **end, const char **start, int *sz)
 {
-       char             c, term;
-       int              i, rlim;
-       const char      *cp, *rstart;
+       const char      *local_start;
+       int              local_sz;
+       char             term;
        enum mandoc_esc  gly; 
 
-       cp = *end;
-       rstart = cp;
-       if (start)
-               *start = rstart;
-       i = rlim = 0;
+       /*
+        * When the caller doesn't provide return storage,
+        * use local storage.
+        */
+
+       if (NULL == start)
+               start = &local_start;
+       if (NULL == sz)
+               sz = &local_sz;
+
+       /*
+        * Beyond the backslash, at least one input character
+        * is part of the escape sequence.  With one exception
+        * (see below), that character won't be returned.
+        */
+
        gly = ESCAPE_ERROR;
+       *start = ++*end;
+       *sz = 0;
        term = '\0';
 
-       switch ((c = cp[i++])) {
+       switch ((*start)[-1]) {
        /*
         * First the glyphs.  There are several different forms of
         * these, but each eventually returns a substring of the glyph
@@ -63,7 +76,7 @@ mandoc_escape(const char **end, const char **start, int *sz)
         */
        case ('('):
                gly = ESCAPE_SPECIAL;
-               rlim = 2;
+               *sz = 2;
                break;
        case ('['):
                gly = ESCAPE_SPECIAL;
@@ -73,14 +86,15 @@ mandoc_escape(const char **end, const char **start, int *sz)
                 * Unicode codepoint.  Here, however, only check whether
                 * it's not a zero-width escape.
                 */
-               if ('u' == cp[i] && ']' != cp[i + 1])
+               if ('u' == (*start)[0] && ']' != (*start)[1])
                        gly = ESCAPE_UNICODE;
                term = ']';
                break;
        case ('C'):
-               if ('\'' != cp[i])
+               if ('\'' != **start)
                        return(ESCAPE_ERROR);
                gly = ESCAPE_SPECIAL;
+               *start = ++*end;
                term = '\'';
                break;
 
@@ -91,7 +105,6 @@ mandoc_escape(const char **end, const char **start, int *sz)
         * let us just skip the next character.
         */
        case ('z'):
-               (*end)++;
                return(ESCAPE_SKIPCHAR);
 
        /*
@@ -118,21 +131,17 @@ mandoc_escape(const char **end, const char **start, int *sz)
        case ('f'):
                if (ESCAPE_ERROR == gly)
                        gly = ESCAPE_FONT;
-
-               rstart= &cp[i];
-               if (start) 
-                       *start = rstart;
-
-               switch (cp[i++]) {
+               switch (**start) {
                case ('('):
-                       rlim = 2;
+                       *start = ++*end;
+                       *sz = 2;
                        break;
                case ('['):
+                       *start = ++*end;
                        term = ']';
                        break;
                default:
-                       rlim = 1;
-                       i--;
+                       *sz = 1;
                        break;
                }
                break;
@@ -154,9 +163,10 @@ mandoc_escape(const char **end, const char **start, int *sz)
        case ('X'):
                /* FALLTHROUGH */
        case ('Z'):
-               if ('\'' != cp[i++])
+               if ('\'' != **start)
                        return(ESCAPE_ERROR);
                gly = ESCAPE_IGNORE;
+               *start = ++*end;
                term = '\'';
                break;
 
@@ -182,10 +192,11 @@ mandoc_escape(const char **end, const char **start, int *sz)
        case ('w'):
                /* FALLTHROUGH */
        case ('x'):
+               if ('\'' != **start)
+                       return(ESCAPE_ERROR);
                if (ESCAPE_ERROR == gly)
                        gly = ESCAPE_IGNORE;
-               if ('\'' != cp[i++])
-                       return(ESCAPE_ERROR);
+               *start = ++*end;
                term = '\'';
                break;
 
@@ -194,17 +205,17 @@ mandoc_escape(const char **end, const char **start, int *sz)
         * XXX Do any other escapes need similar handling?
         */
        case ('N'):
-               if ('\0' == cp[i])
+               if ('\0' == **start)
                        return(ESCAPE_ERROR);
-               *end = &cp[++i];
-               if (isdigit((unsigned char)cp[i-1]))
+               (*end)++;
+               if (isdigit((unsigned char)**start)) {
+                       *sz = 1;
                        return(ESCAPE_IGNORE);
+               }
+               (*start)++;
                while (isdigit((unsigned char)**end))
                        (*end)++;
-               if (start)
-                       *start = &cp[i];
-               if (sz)
-                       *sz = *end - &cp[i];
+               *sz = *end - *start;
                if ('\0' != **end)
                        (*end)++;
                return(ESCAPE_NUMBERED);
@@ -215,54 +226,43 @@ mandoc_escape(const char **end, const char **start, int *sz)
        case ('s'):
                gly = ESCAPE_IGNORE;
 
-               rstart = &cp[i];
-               if (start) 
-                       *start = rstart;
-
                /* See +/- counts as a sign. */
-               c = cp[i];
-               if ('+' == c || '-' == c || ASCII_HYPH == c)
-                       ++i;
+               if ('+' == **end || '-' == **end || ASCII_HYPH == **end)
+                       (*end)++;
 
-               switch (cp[i++]) {
+               switch (**end) {
                case ('('):
-                       rlim = 2;
+                       *start = ++*end;
+                       *sz = 2;
                        break;
                case ('['):
+                       *start = ++*end;
                        term = ']';
                        break;
                case ('\''):
+                       *start = ++*end;
                        term = '\'';
                        break;
                default:
-                       rlim = 1;
-                       i--;
+                       *sz = 1;
                        break;
                }
 
-               /* See +/- counts as a sign. */
-               c = cp[i];
-               if ('+' == c || '-' == c || ASCII_HYPH == c)
-                       ++i;
-
                break;
 
        /*
         * Anything else is assumed to be a glyph.
+        * In this case, pass back the character after the backslash.
         */
        default:
                gly = ESCAPE_SPECIAL;
-               rlim = 1;
-               i--;
+               *start = --*end;
+               *sz = 1;
                break;
        }
 
        assert(ESCAPE_ERROR != gly);
 
-       *end = rstart = &cp[i];
-       if (start)
-               *start = rstart;
-
        /*
         * Read up to the terminating character,
         * paying attention to nested escapes.
@@ -284,15 +284,13 @@ mandoc_escape(const char **end, const char **start, int *sz)
                                break;
                        }
                }
-               rlim = (*end)++ - rstart;
+               *sz = (*end)++ - *start;
        } else {
-               assert(rlim > 0);
-               if ((size_t)rlim > strlen(rstart))
+               assert(*sz > 0);
+               if ((size_t)*sz > strlen(*start))
                        return(ESCAPE_ERROR);
-               *end += rlim;
+               *end += *sz;
        }
-       if (sz)
-               *sz = rlim;
 
        /* Run post-processors. */
 
@@ -302,12 +300,13 @@ mandoc_escape(const char **end, const char **start, int *sz)
                 * Pretend that the constant-width font modes are the
                 * same as the regular font modes.
                 */
-               if (2 == rlim && 'C' == *rstart)
-                       rstart++;
-               else if (1 != rlim)
+               if (2 == *sz && 'C' == **start) {
+                       (*start)++;
+                       (*sz)--;
+               } else if (1 != *sz)
                        break;
 
-               switch (*rstart) {
+               switch (**start) {
                case ('3'):
                        /* FALLTHROUGH */
                case ('B'):
@@ -329,9 +328,7 @@ mandoc_escape(const char **end, const char **start, int *sz)
                }
                break;
        case (ESCAPE_SPECIAL):
-               if (1 != rlim)
-                       break;
-               if ('c' == *rstart)
+               if (1 == *sz && 'c' == **start)
                        gly = ESCAPE_NOSPACE;
                break;
        default:
@@ -606,32 +603,6 @@ mandoc_eos(const char *p, size_t sz, int enclosed)
        return(found && !enclosed);
 }
 
-/*
- * Find out whether a line is a macro line or not.  If it is, adjust the
- * current position and return one; if it isn't, return zero and don't
- * change the current position.
- */
-int
-mandoc_getcontrol(const char *cp, int *ppos)
-{
-       int             pos;
-
-       pos = *ppos;
-
-       if ('\\' == cp[pos] && '.' == cp[pos + 1])
-               pos += 2;
-       else if ('.' == cp[pos] || '\'' == cp[pos])
-               pos++;
-       else
-               return(0);
-
-       while (' ' == cp[pos] || '\t' == cp[pos])
-               pos++;
-
-       *ppos = pos;
-       return(1);
-}
-
 /*
  * Convert a string to a long that may not be <0.
  * If the string is invalid, or is less than 0, return -1.