]> git.cameronkatri.com Git - mandoc.git/blobdiff - roff.c
Fix a TODO to the effect that `.if n \{\ foo .br \}' was failing due to
[mandoc.git] / roff.c
diff --git a/roff.c b/roff.c
index 7e3af4c3fd019ca8cc6695bfd337a37b5ae99b2b..35e0f883ef74eaa9ed41ab20c9eed25b2775b85f 100644 (file)
--- a/roff.c
+++ b/roff.c
@@ -1,4 +1,4 @@
-/*     $Id: roff.c,v 1.127 2011/03/15 16:23:51 kristaps Exp $ */
+/*     $Id: roff.c,v 1.139 2011/05/24 14:00:39 kristaps Exp $ */
 /*
  * Copyright (c) 2010, 2011 Kristaps Dzonsons <kristaps@bsd.lv>
  * Copyright (c) 2010, 2011 Ingo Schwarze <schwarze@openbsd.org>
 #endif
 
 #include <assert.h>
-#include <errno.h>
 #include <ctype.h>
-#include <limits.h>
 #include <stdlib.h>
 #include <string.h>
-#include <stdio.h>
 
 #include "mandoc.h"
-#include "roff.h"
 #include "libroff.h"
 #include "libmandoc.h"
 
 #define        RSTACK_MAX      128
 
-#define        ROFF_CTL(c) \
-       ('.' == (c) || '\'' == (c))
-
 enum   rofft {
        ROFF_ad,
        ROFF_am,
@@ -84,9 +77,8 @@ struct        roffstr {
 };
 
 struct roff {
+       struct mparse   *parse; /* parse point */
        struct roffnode *last; /* leaf of stack */
-       mandocmsg        msg; /* err/warn/fatal messages */
-       void            *data; /* privdata for messages */
        enum roffrule    rstack[RSTACK_MAX]; /* stack of !`ie' rules */
        int              rstackpos; /* position in rstack */
        struct regset   *regs; /* read/writable registers */
@@ -211,7 +203,6 @@ static      void             roffnode_push(struct roff *, enum rofft,
                                const char *, int, int);
 static void             roffnode_pop(struct roff *);
 static enum rofft       roff_parse(struct roff *, const char *, int *);
-static int              roff_parse_nat(const char *, unsigned int *);
 
 /* See roff_hash_find() */
 #define        ROFF_HASH(p)    (p[0] - ASCII_LO)
@@ -282,10 +273,6 @@ roffnode_pop(struct roff *r)
        assert(r->last);
        p = r->last; 
 
-       if (ROFF_el == p->tok)
-               if (r->rstackpos > -1)
-                       r->rstackpos--;
-
        r->last = r->last->parent;
        free(p->name);
        free(p->end);
@@ -361,14 +348,13 @@ roff_free(struct roff *r)
 
 
 struct roff *
-roff_alloc(struct regset *regs, void *data, const mandocmsg msg)
+roff_alloc(struct regset *regs, struct mparse *parse)
 {
        struct roff     *r;
 
        r = mandoc_calloc(1, sizeof(struct roff));
        r->regs = regs;
-       r->msg = msg;
-       r->data = data;
+       r->parse = parse;
        r->rstackpos = -1;
        
        roff_hash_init();
@@ -479,7 +465,7 @@ roff_parseln(struct roff *r, int ln, char **bufp,
 {
        enum rofft       t;
        enum rofferr     e;
-       int              ppos;
+       int              ppos, ctl;
 
        /*
         * Run the reserved-word filter only if we have some reserved
@@ -489,6 +475,9 @@ roff_parseln(struct roff *r, int ln, char **bufp,
        if (r->first_string && ! roff_res(r, bufp, szp, pos))
                return(ROFF_REPARSE);
 
+       ppos = pos;
+       ctl = mandoc_getcontrol(*bufp, &pos);
+
        /*
         * First, if a scope is open and we're not a macro, pass the
         * text through the macro's filter.  If a scope isn't open and
@@ -497,7 +486,7 @@ roff_parseln(struct roff *r, int ln, char **bufp,
         * no matter our state.
         */
 
-       if (r->last && ! ROFF_CTL((*bufp)[pos])) {
+       if (r->last && ! ctl) {
                t = r->last->tok;
                assert(roffs[t].text);
                e = (*roffs[t].text)
@@ -506,18 +495,18 @@ roff_parseln(struct roff *r, int ln, char **bufp,
                if (ROFF_CONT != e)
                        return(e);
                if (r->eqn)
-                       return(eqn_read(&r->eqn, ln, *bufp, *offs));
+                       return(eqn_read(&r->eqn, ln, *bufp, pos));
                if (r->tbl)
-                       return(tbl_read(r->tbl, ln, *bufp, *offs));
+                       return(tbl_read(r->tbl, ln, *bufp, pos));
                return(ROFF_CONT);
-       } else if ( ! ROFF_CTL((*bufp)[pos])) {
+       } else if ( ! ctl) {
                if (r->eqn)
-                       return(eqn_read(&r->eqn, ln, *bufp, *offs));
+                       return(eqn_read(&r->eqn, ln, *bufp, pos));
                if (r->tbl)
-                       return(tbl_read(r->tbl, ln, *bufp, *offs));
+                       return(tbl_read(r->tbl, ln, *bufp, pos));
                return(ROFF_CONT);
        } else if (r->eqn)
-               return(eqn_read(&r->eqn, ln, *bufp, *offs));
+               return(eqn_read(&r->eqn, ln, *bufp, ppos));
 
        /*
         * If a scope is open, go to the child handler for that macro,
@@ -530,7 +519,7 @@ roff_parseln(struct roff *r, int ln, char **bufp,
                assert(roffs[t].sub);
                return((*roffs[t].sub)
                                (r, t, bufp, szp, 
-                                ln, pos, pos, offs));
+                                ln, ppos, pos, offs));
        }
 
        /*
@@ -539,7 +528,6 @@ roff_parseln(struct roff *r, int ln, char **bufp,
         * the compilers handle it.
         */
 
-       ppos = pos;
        if (ROFF_MAX == (t = roff_parse(r, *bufp, &pos)))
                return(ROFF_CONT);
 
@@ -555,25 +543,24 @@ roff_endparse(struct roff *r)
 {
 
        if (r->last)
-               (*r->msg)(MANDOCERR_SCOPEEXIT, r->data,
+               mandoc_msg(MANDOCERR_SCOPEEXIT, r->parse,
                                r->last->line, r->last->col, NULL);
 
        if (r->eqn) {
-               (*r->msg)(MANDOCERR_SCOPEEXIT, r->data
+               mandoc_msg(MANDOCERR_SCOPEEXIT, r->parse
                                r->eqn->eqn.line, r->eqn->eqn.pos, NULL);
                eqn_end(r->eqn);
                r->eqn = NULL;
        }
 
        if (r->tbl) {
-               (*r->msg)(MANDOCERR_SCOPEEXIT, r->data
+               mandoc_msg(MANDOCERR_SCOPEEXIT, r->parse
                                r->tbl->line, r->tbl->pos, NULL);
                tbl_end(r->tbl);
                r->tbl = NULL;
        }
 }
 
-
 /*
  * Parse a roff node's type from the input buffer.  This must be in the
  * form of ".foo xxx" in the usual way.
@@ -585,13 +572,7 @@ roff_parse(struct roff *r, const char *buf, int *pos)
        size_t           maclen;
        enum rofft       t;
 
-       assert(ROFF_CTL(buf[*pos]));
-       (*pos)++;
-
-       while (' ' == buf[*pos] || '\t' == buf[*pos])
-               (*pos)++;
-
-       if ('\0' == buf[*pos])
+       if ('\0' == buf[*pos] || '"' == buf[*pos])
                return(ROFF_MAX);
 
        mac = buf + *pos;
@@ -601,33 +582,13 @@ roff_parse(struct roff *r, const char *buf, int *pos)
            ? ROFF_USERDEF : roff_hash_find(mac, maclen);
 
        *pos += (int)maclen;
+
        while (buf[*pos] && ' ' == buf[*pos])
                (*pos)++;
 
        return(t);
 }
 
-
-static int
-roff_parse_nat(const char *buf, unsigned int *res)
-{
-       char            *ep;
-       long             lval;
-
-       errno = 0;
-       lval = strtol(buf, &ep, 10);
-       if (buf[0] == '\0' || *ep != '\0')
-               return(0);
-       if ((errno == ERANGE && 
-                       (lval == LONG_MAX || lval == LONG_MIN)) ||
-                       (lval > INT_MAX || lval < 0))
-               return(0);
-
-       *res = (unsigned int)lval;
-       return(1);
-}
-
-
 /* ARGSUSED */
 static enum rofferr
 roff_cblock(ROFF_ARGS)
@@ -639,7 +600,7 @@ roff_cblock(ROFF_ARGS)
         */
 
        if (NULL == r->last) {
-               (*r->msg)(MANDOCERR_NOSCOPE, r->data, ln, ppos, NULL);
+               mandoc_msg(MANDOCERR_NOSCOPE, r->parse, ln, ppos, NULL);
                return(ROFF_IGN);
        }
 
@@ -658,12 +619,12 @@ roff_cblock(ROFF_ARGS)
        case (ROFF_ig):
                break;
        default:
-               (*r->msg)(MANDOCERR_NOSCOPE, r->data, ln, ppos, NULL);
+               mandoc_msg(MANDOCERR_NOSCOPE, r->parse, ln, ppos, NULL);
                return(ROFF_IGN);
        }
 
        if ((*bufp)[pos])
-               (*r->msg)(MANDOCERR_ARGSLOST, r->data, ln, pos, NULL);
+               mandoc_msg(MANDOCERR_ARGSLOST, r->parse, ln, pos, NULL);
 
        roffnode_pop(r);
        roffnode_cleanscope(r);
@@ -690,7 +651,7 @@ roff_ccond(ROFF_ARGS)
 {
 
        if (NULL == r->last) {
-               (*r->msg)(MANDOCERR_NOSCOPE, r->data, ln, ppos, NULL);
+               mandoc_msg(MANDOCERR_NOSCOPE, r->parse, ln, ppos, NULL);
                return(ROFF_IGN);
        }
 
@@ -702,17 +663,17 @@ roff_ccond(ROFF_ARGS)
        case (ROFF_if):
                break;
        default:
-               (*r->msg)(MANDOCERR_NOSCOPE, r->data, ln, ppos, NULL);
+               mandoc_msg(MANDOCERR_NOSCOPE, r->parse, ln, ppos, NULL);
                return(ROFF_IGN);
        }
 
        if (r->last->endspan > -1) {
-               (*r->msg)(MANDOCERR_NOSCOPE, r->data, ln, ppos, NULL);
+               mandoc_msg(MANDOCERR_NOSCOPE, r->parse, ln, ppos, NULL);
                return(ROFF_IGN);
        }
 
        if ((*bufp)[pos])
-               (*r->msg)(MANDOCERR_ARGSLOST, r->data, ln, pos, NULL);
+               mandoc_msg(MANDOCERR_ARGSLOST, r->parse, ln, pos, NULL);
 
        roffnode_pop(r);
        roffnode_cleanscope(r);
@@ -732,7 +693,7 @@ roff_block(ROFF_ARGS)
 
        if (ROFF_ig != tok) {
                if ('\0' == (*bufp)[pos]) {
-                       (*r->msg)(MANDOCERR_NOARGS, r->data, ln, ppos, NULL);
+                       mandoc_msg(MANDOCERR_NOARGS, r->parse, ln, ppos, NULL);
                        return(ROFF_IGN);
                }
 
@@ -746,13 +707,13 @@ roff_block(ROFF_ARGS)
                if (ROFF_de == tok)
                        name = *bufp + pos;
                else
-                       (*r->msg)(MANDOCERR_REQUEST, r->data, ln, ppos,
+                       mandoc_msg(MANDOCERR_REQUEST, r->parse, ln, ppos,
                            roffs[tok].name);
 
-               while ((*bufp)[pos] && ' ' != (*bufp)[pos])
+               while ((*bufp)[pos] && ! isspace((unsigned char)(*bufp)[pos]))
                        pos++;
 
-               while (' ' == (*bufp)[pos])
+               while (isspace((unsigned char)(*bufp)[pos]))
                        (*bufp)[pos++] = '\0';
        }
 
@@ -773,9 +734,7 @@ roff_block(ROFF_ARGS)
        /* If present, process the custom end-of-line marker. */
 
        sv = pos;
-       while ((*bufp)[pos] &&
-                       ' ' != (*bufp)[pos] && 
-                       '\t' != (*bufp)[pos])
+       while ((*bufp)[pos] && ! isspace((unsigned char)(*bufp)[pos]))
                pos++;
 
        /*
@@ -796,7 +755,7 @@ roff_block(ROFF_ARGS)
        r->last->end[(int)sz] = '\0';
 
        if ((*bufp)[pos])
-               (*r->msg)(MANDOCERR_ARGSLOST, r->data, ln, pos, NULL);
+               mandoc_msg(MANDOCERR_ARGSLOST, r->parse, ln, pos, NULL);
 
        return(ROFF_IGN);
 }
@@ -819,11 +778,7 @@ roff_block_sub(ROFF_ARGS)
         */
 
        if (r->last->end) {
-               i = pos + 1;
-               while (' ' == (*bufp)[i] || '\t' == (*bufp)[i])
-                       i++;
-
-               for (j = 0; r->last->end[j]; j++, i++)
+               for (i = pos, j = 0; r->last->end[j]; j++, i++)
                        if ((*bufp)[i] != r->last->end[j])
                                break;
 
@@ -834,6 +789,10 @@ roff_block_sub(ROFF_ARGS)
                        roffnode_pop(r);
                        roffnode_cleanscope(r);
 
+                       while (' ' == (*bufp)[i] || '\t' == (*bufp)[i])
+                               i++;
+
+                       pos = i;
                        if (ROFF_MAX != roff_parse(r, *bufp, &pos))
                                return(ROFF_RERUN);
                        return(ROFF_IGN);
@@ -845,7 +804,6 @@ roff_block_sub(ROFF_ARGS)
         * pulling it out of the hashtable.
         */
 
-       ppos = pos;
        t = roff_parse(r, *bufp, &pos);
 
        /*
@@ -882,22 +840,34 @@ roff_cond_sub(ROFF_ARGS)
 {
        enum rofft       t;
        enum roffrule    rr;
+       char            *ep;
 
-       ppos = pos;
        rr = r->last->rule;
+       roffnode_cleanscope(r);
 
-       /* 
-        * Clean out scope.  If we've closed ourselves, then don't
-        * continue. 
+       /*
+        * If the macro is unknown, first check if it contains a closing
+        * delimiter `\}'.  If it does, close out our scope and return
+        * the currently-scoped rule (ignore or continue).  Else, drop
+        * into the currently-scoped rule.
         */
 
-       roffnode_cleanscope(r);
-
        if (ROFF_MAX == (t = roff_parse(r, *bufp, &pos))) {
-               if ('\\' == (*bufp)[pos] && '}' == (*bufp)[pos + 1])
-                       return(roff_ccond
-                               (r, ROFF_ccond, bufp, szp,
-                                ln, pos, pos + 2, offs));
+               /*
+                * Jump through hoops to detect a \}, because it could
+                * be (say) \\}, which is something completely
+                * different.
+                */
+               ep = &(*bufp)[pos];
+               for ( ; NULL != (ep = strchr(ep, '\\')); ep++) {
+                       ep++;
+                       if ('}' != *ep)
+                               continue;
+                       *--ep = '\0';
+                       roff_ccond(r, ROFF_ccond, bufp, szp, 
+                                       ln, pos, pos + 2, offs);
+                       break;
+               }
                return(ROFFRULE_DENY == rr ? ROFF_IGN : ROFF_CONT);
        }
 
@@ -906,6 +876,7 @@ roff_cond_sub(ROFF_ARGS)
         * if they're either structurally required (such as loops and
         * conditionals) or a closing macro.
         */
+
        if (ROFFRULE_DENY == rr)
                if ( ! (ROFFMAC_STRUCT & roffs[t].flags))
                        if (ROFF_ccond != t)
@@ -977,7 +948,7 @@ roff_line_ignore(ROFF_ARGS)
 {
 
        if (ROFF_it == tok)
-               (*r->msg)(MANDOCERR_REQUEST, r->data, ln, ppos, "it");
+               mandoc_msg(MANDOCERR_REQUEST, r->parse, ln, ppos, "it");
 
        return(ROFF_IGN);
 }
@@ -989,29 +960,20 @@ roff_cond(ROFF_ARGS)
        int              sv;
        enum roffrule    rule;
 
-       /* Stack overflow! */
-
-       if (ROFF_ie == tok && r->rstackpos == RSTACK_MAX - 1) {
-               (*r->msg)(MANDOCERR_MEM, r->data, ln, ppos, NULL);
-               return(ROFF_ERR);
-       }
-
-       /* First, evaluate the conditional. */
+       /* 
+        * An `.el' has no conditional body: it will consume the value
+        * of the current rstack entry set in prior `ie' calls or
+        * defaults to DENY.  
+        *
+        * If we're not an `el', however, then evaluate the conditional.
+        */
 
-       if (ROFF_el == tok) {
-               /* 
-                * An `.el' will get the value of the current rstack
-                * entry set in prior `ie' calls or defaults to DENY.
-                */
-               if (r->rstackpos < 0)
-                       rule = ROFFRULE_DENY;
-               else
-                       rule = r->rstack[r->rstackpos];
-       } else
-               rule = roff_evalcond(*bufp, &pos);
+       rule = ROFF_el == tok ?
+               (r->rstackpos < 0 ? 
+                ROFFRULE_DENY : r->rstack[r->rstackpos--]) :
+               roff_evalcond(*bufp, &pos);
 
        sv = pos;
-
        while (' ' == (*bufp)[pos])
                pos++;
 
@@ -1023,7 +985,7 @@ roff_cond(ROFF_ARGS)
         */
 
        if ('\0' == (*bufp)[pos] && sv != pos) {
-               (*r->msg)(MANDOCERR_NOARGS, r->data, ln, ppos, NULL);
+               mandoc_msg(MANDOCERR_NOARGS, r->parse, ln, ppos, NULL);
                return(ROFF_IGN);
        }
 
@@ -1031,16 +993,20 @@ roff_cond(ROFF_ARGS)
 
        r->last->rule = rule;
 
+       /*
+        * An if-else will put the NEGATION of the current evaluated
+        * conditional into the stack of rules.
+        */
+
        if (ROFF_ie == tok) {
-               /*
-                * An if-else will put the NEGATION of the current
-                * evaluated conditional into the stack.
-                */
-               r->rstackpos++;
-               if (ROFFRULE_DENY == r->last->rule)
-                       r->rstack[r->rstackpos] = ROFFRULE_ALLOW;
-               else
-                       r->rstack[r->rstackpos] = ROFFRULE_DENY;
+               if (r->rstackpos == RSTACK_MAX - 1) {
+                       mandoc_msg(MANDOCERR_MEM, 
+                               r->parse, ln, ppos, NULL);
+                       return(ROFF_ERR);
+               }
+               r->rstack[++r->rstackpos] = 
+                       ROFFRULE_DENY == r->last->rule ?
+                       ROFFRULE_ALLOW : ROFFRULE_DENY;
        }
 
        /* If the parent has false as its rule, then so do we. */
@@ -1113,6 +1079,7 @@ roff_nr(ROFF_ARGS)
 {
        const char      *key;
        char            *val;
+       int              iv;
        struct reg      *rg;
 
        val = *bufp + pos;
@@ -1121,8 +1088,10 @@ roff_nr(ROFF_ARGS)
 
        if (0 == strcmp(key, "nS")) {
                rg[(int)REG_nS].set = 1;
-               if ( ! roff_parse_nat(val, &rg[(int)REG_nS].v.u))
-                       rg[(int)REG_nS].v.u = 0;
+               if ((iv = mandoc_strntou(val, strlen(val), 10)) >= 0)
+                       rg[REG_nS].v.u = (unsigned)iv;
+               else
+                       rg[(int)REG_nS].v.u = 0u;
        }
 
        return(ROFF_IGN);
@@ -1150,7 +1119,7 @@ roff_TE(ROFF_ARGS)
 {
 
        if (NULL == r->tbl)
-               (*r->msg)(MANDOCERR_NOSCOPE, r->data, ln, ppos, NULL);
+               mandoc_msg(MANDOCERR_NOSCOPE, r->parse, ln, ppos, NULL);
        else
                tbl_end(r->tbl);
 
@@ -1164,7 +1133,7 @@ roff_T_(ROFF_ARGS)
 {
 
        if (NULL == r->tbl)
-               (*r->msg)(MANDOCERR_NOSCOPE, r->data, ln, ppos, NULL);
+               mandoc_msg(MANDOCERR_NOSCOPE, r->parse, ln, ppos, NULL);
        else
                tbl_restart(ppos, ln, r->tbl);
 
@@ -1194,7 +1163,7 @@ static enum rofferr
 roff_EN(ROFF_ARGS)
 {
 
-       (*r->msg)(MANDOCERR_NOSCOPE, r->data, ln, ppos, NULL);
+       mandoc_msg(MANDOCERR_NOSCOPE, r->parse, ln, ppos, NULL);
        return(ROFF_IGN);
 }
 
@@ -1205,11 +1174,11 @@ roff_TS(ROFF_ARGS)
        struct tbl_node *t;
 
        if (r->tbl) {
-               (*r->msg)(MANDOCERR_SCOPEBROKEN, r->data, ln, ppos, NULL);
+               mandoc_msg(MANDOCERR_SCOPEBROKEN, r->parse, ln, ppos, NULL);
                tbl_end(r->tbl);
        }
 
-       t = tbl_alloc(ppos, ln, r->data, r->msg);
+       t = tbl_alloc(ppos, ln, r->parse);
 
        if (r->last_tbl)
                r->last_tbl->next = t;
@@ -1226,7 +1195,7 @@ roff_so(ROFF_ARGS)
 {
        char *name;
 
-       (*r->msg)(MANDOCERR_SO, r->data, ln, ppos, NULL);
+       mandoc_msg(MANDOCERR_SO, r->parse, ln, ppos, NULL);
 
        /*
         * Handle `so'.  Be EXTREMELY careful, as we shouldn't be
@@ -1237,7 +1206,7 @@ roff_so(ROFF_ARGS)
 
        name = *bufp + pos;
        if ('/' == *name || strstr(name, "../") || strstr(name, "/..")) {
-               (*r->msg)(MANDOCERR_SOPATH, r->data, ln, pos, NULL);
+               mandoc_msg(MANDOCERR_SOPATH, r->parse, ln, pos, NULL);
                return(ROFF_ERR);
        }
 
@@ -1260,7 +1229,7 @@ roff_userdef(ROFF_ARGS)
        cp = *bufp + pos;
        for (i = 0; i < 9; i++)
                arg[i] = '\0' == *cp ? "" :
-                   mandoc_getarg(&cp, r->msg, r->data, ln, &pos);
+                   mandoc_getarg(r->parse, &cp, ln, &pos);
 
        /*
         * Expand macro arguments.
@@ -1316,7 +1285,7 @@ roff_getname(struct roff *r, char **cpp, int ln, int pos)
                cp++;
                if ('\\' == *cp)
                        continue;
-               (*r->msg)(MANDOCERR_NAMESC, r->data, ln, pos, NULL);
+               mandoc_msg(MANDOCERR_NAMESC, r->parse, ln, pos, NULL);
                *cp = '\0';
                name = cp;
        }