Remove delims from struct tbl (not used anywhere and never will be).

[mandoc.git] / roff.c

diff --git a/roff.c b/roff.c
index 5531dc8603d6869e8b4fbcc45cc48768da25855e..5053bef51401ff993472d83e139030327ea07a0a 100644 (file)
--- a/roff.c
+++ b/roff.c
@@ -1,7 +1,7 @@
-/*     $Id: roff.c,v 1.114 2010/12/31 18:19:43 kristaps Exp $ */
+/*     $Id: roff.c,v 1.120 2011/01/03 23:24:16 schwarze Exp $ */
 /*
- * Copyright (c) 2010 Kristaps Dzonsons <kristaps@bsd.lv>
- * Copyright (c) 2010 Ingo Schwarze <schwarze@openbsd.org>
+ * Copyright (c) 2010, 2011 Kristaps Dzonsons <kristaps@bsd.lv>
+ * Copyright (c) 2010, 2011 Ingo Schwarze <schwarze@openbsd.org>
  *
  * Permission to use, copy, modify, and distribute this software for any
  * purpose with or without fee is hereby granted, provided that the above
@@ -86,9 +86,9 @@ struct        roff {
        struct regset   *regs; /* read/writable registers */
        struct roffstr  *first_string; /* user-defined strings & macros */
        const char      *current_string; /* value of last called user macro */
-       struct tbl      *first_tbl; /* first table parsed */
-       struct tbl      *last_tbl; /* last table parsed */
-       struct tbl      *tbl; /* current table being parsed */
+       struct tbl_node *first_tbl; /* first table parsed */
+       struct tbl_node *last_tbl; /* last table parsed */
+       struct tbl_node *tbl; /* current table being parsed */
 };
 
 struct roffnode {
@@ -301,7 +301,7 @@ roffnode_push(struct roff *r, enum rofft tok, const char *name,
 static void
 roff_free1(struct roff *r)
 {
-       struct tbl      *t;
+       struct tbl_node *t;
 
        while (r->first_tbl) {
                t = r->first_tbl;
@@ -515,15 +515,20 @@ roff_parseln(struct roff *r, int ln, char **bufp,
 }
 
 
-int
+void
 roff_endparse(struct roff *r)
 {
 
-       /* FIXME: if r->tbl */
        if (r->last)
                (*r->msg)(MANDOCERR_SCOPEEXIT, r->data, 
                                r->last->line, r->last->col, NULL);
-       return(1);
+
+       if (r->tbl) {
+               (*r->msg)(MANDOCERR_SCOPEEXIT, r->data, 
+                               r->tbl->line, r->tbl->pos, NULL);
+               tbl_end(r->tbl);
+               r->tbl = NULL;
+       }
 }
 
 
@@ -1123,6 +1128,8 @@ roff_TE(ROFF_ARGS)
 
        if (NULL == r->tbl)
                (*r->msg)(MANDOCERR_NOSCOPE, r->data, ln, ppos, NULL);
+       else
+               tbl_end(r->tbl);
 
        r->tbl = NULL;
        return(ROFF_IGN);
@@ -1136,7 +1143,7 @@ roff_T_(ROFF_ARGS)
        if (NULL == r->tbl)
                (*r->msg)(MANDOCERR_NOSCOPE, r->data, ln, ppos, NULL);
        else
-               tbl_restart(r->tbl);
+               tbl_restart(ppos, ln, r->tbl);
 
        return(ROFF_IGN);
 }
@@ -1145,12 +1152,14 @@ roff_T_(ROFF_ARGS)
 static enum rofferr
 roff_TS(ROFF_ARGS)
 {
-       struct tbl      *t;
+       struct tbl_node *t;
 
-       if (r->tbl)
+       if (r->tbl) {
                (*r->msg)(MANDOCERR_SCOPEBROKEN, r->data, ln, ppos, NULL);
+               tbl_end(r->tbl);
+       }
 
-       t = tbl_alloc(r->data, r->msg);
+       t = tbl_alloc(ppos, ln, r->data, r->msg);
 
        if (r->last_tbl)
                r->last_tbl->next = t;
@@ -1192,53 +1201,16 @@ roff_userdef(ROFF_ARGS)
 {
        const char       *arg[9];
        char             *cp, *n1, *n2;
-       int               i, quoted, pairs;
+       int               i;
 
        /*
         * Collect pointers to macro argument strings
         * and null-terminate them.
         */
        cp = *bufp + pos;
-       for (i = 0; i < 9; i++) {
-               /* Quoting can only start with a new word. */
-               if ('"' == *cp) {
-                       quoted = 1;
-                       cp++;
-               } else
-                       quoted = 0;
-               arg[i] = cp;
-               for (pairs = 0; '\0' != *cp; cp++) {
-                       /* Unquoted arguments end at blanks. */
-                       if (0 == quoted) {
-                               if (' ' == *cp)
-                                       break;
-                               continue;
-                       }
-                       /* After pairs of quotes, move left. */
-                       if (pairs)
-                               cp[-pairs] = cp[0];
-                       /* Pairs of quotes do not end words, ... */
-                       if ('"' == cp[0] && '"' == cp[1]) {
-                               pairs++;
-                               cp++;
-                               continue;
-                       }
-                       /* ... but solitary quotes do. */
-                       if ('"' != *cp)
-                               continue;
-                       if (pairs)
-                               cp[-pairs] = '\0';
-                       *cp = ' ';
-                       break;
-               }
-               /* Last argument; the remaining ones are empty strings. */
-               if ('\0' == *cp)
-                       continue;
-               /* Null-terminate argument and move to the next one. */
-               *cp++ = '\0';
-               while (' ' == *cp)
-                       cp++;
-       }
+       for (i = 0; i < 9; i++)
+               arg[i] = '\0' == *cp ? "" :
+                   mandoc_getarg(&cp, r->msg, r->data, ln, &pos);
 
        /*
         * Expand macro arguments.