]> git.cameronkatri.com Git - mandoc.git/blobdiff - mdoc.c
Tiny edit required after MDOC_HALT change.
[mandoc.git] / mdoc.c
diff --git a/mdoc.c b/mdoc.c
index 8e46ff4995c148d3b0932a7e6947c6f90b528336..af3952172099b7f2e85ae05c1c40905cdd5d276f 100644 (file)
--- a/mdoc.c
+++ b/mdoc.c
@@ -1,4 +1,4 @@
-/*     $Id: mdoc.c,v 1.170 2010/12/16 17:14:48 kristaps Exp $ */
+/*     $Id: mdoc.c,v 1.177 2011/01/03 11:27:33 kristaps Exp $ */
 /*
  * Copyright (c) 2008, 2009, 2010 Kristaps Dzonsons <kristaps@bsd.lv>
  * Copyright (c) 2010 Ingo Schwarze <schwarze@openbsd.org>
@@ -98,13 +98,16 @@ static      int               node_append(struct mdoc *,
                                struct mdoc_node *);
 static int               mdoc_ptext(struct mdoc *, int, char *, int);
 static int               mdoc_pmacro(struct mdoc *, int, char *, int);
+static int               mdoc_span_alloc(struct mdoc *, 
+                               const struct tbl_span *);
 
 
 const struct mdoc_node *
 mdoc_node(const struct mdoc *m)
 {
 
-       return(MDOC_HALT & m->flags ? NULL : m->first);
+       assert( ! (MDOC_HALT & m->flags));
+       return(m->first);
 }
 
 
@@ -112,7 +115,8 @@ const struct mdoc_meta *
 mdoc_meta(const struct mdoc *m)
 {
 
-       return(MDOC_HALT & m->flags ? NULL : &m->meta);
+       assert( ! (MDOC_HALT & m->flags));
+       return(&m->meta);
 }
 
 
@@ -213,14 +217,30 @@ int
 mdoc_endparse(struct mdoc *m)
 {
 
-       if (MDOC_HALT & m->flags)
-               return(0);
-       else if (mdoc_macroend(m))
+       assert( ! (MDOC_HALT & m->flags));
+       if (mdoc_macroend(m))
                return(1);
        m->flags |= MDOC_HALT;
        return(0);
 }
 
+int
+mdoc_addspan(struct mdoc *m, const struct tbl_span *sp)
+{
+
+       assert( ! (MDOC_HALT & m->flags));
+
+       /* No text before an initial macro. */
+
+       if (SEC_NONE == m->lastnamed) {
+               /* FIXME: grab from span. */
+               mdoc_pmsg(m, 0, 0, MANDOCERR_NOTEXT);
+               return(1);
+       }
+
+       return(mdoc_span_alloc(m, sp));
+}
+
 
 /*
  * Main parse routine.  Parses a single line -- really just hands off to
@@ -230,8 +250,7 @@ int
 mdoc_parseln(struct mdoc *m, int ln, char *buf, int offs)
 {
 
-       if (MDOC_HALT & m->flags)
-               return(0);
+       assert( ! (MDOC_HALT & m->flags));
 
        m->flags |= MDOC_NEWLINE;
 
@@ -277,15 +296,16 @@ mdoc_macro(MACRO_PROT_ARGS)
        /* If we're in the body, deny prologue calls. */
 
        if (MDOC_PROLOGUE & mdoc_macros[tok].flags && 
-                       MDOC_PBODY & m->flags)
-               return(mdoc_pmsg(m, line, ppos, MANDOCERR_BADBODY));
+                       MDOC_PBODY & m->flags) {
+               mdoc_pmsg(m, line, ppos, MANDOCERR_BADBODY);
+               return(1);
+       }
 
        /* If we're in the prologue, deny "body" macros.  */
 
        if ( ! (MDOC_PROLOGUE & mdoc_macros[tok].flags) && 
                        ! (MDOC_PBODY & m->flags)) {
-               if ( ! mdoc_pmsg(m, line, ppos, MANDOCERR_BADPROLOG))
-                       return(0);
+               mdoc_pmsg(m, line, ppos, MANDOCERR_BADPROLOG);
                if (NULL == m->meta.msec)
                        m->meta.msec = mandoc_strdup("1");
                if (NULL == m->meta.title)
@@ -328,6 +348,23 @@ node_append(struct mdoc *mdoc, struct mdoc_node *p)
 
        p->parent->nchild++;
 
+       /*
+        * Copy over the normalised-data pointer of our parent.  Not
+        * everybody has one, but copying a null pointer is fine.
+        */
+
+       switch (p->type) {
+       case (MDOC_BODY):
+               /* FALLTHROUGH */
+       case (MDOC_TAIL):
+               /* FALLTHROUGH */
+       case (MDOC_HEAD):
+               p->norm = p->parent->norm;
+               break;
+       default:
+               break;
+       }
+
        if ( ! mdoc_valid_pre(mdoc, p))
                return(0);
 
@@ -353,6 +390,8 @@ node_append(struct mdoc *mdoc, struct mdoc_node *p)
        mdoc->last = p;
 
        switch (p->type) {
+       case (MDOC_TBL):
+               /* FALLTHROUGH */
        case (MDOC_TEXT):
                if ( ! mdoc_valid_post(mdoc))
                        return(0);
@@ -460,6 +499,21 @@ mdoc_block_alloc(struct mdoc *m, int line, int pos,
        p->args = args;
        if (p->args)
                (args->refcnt)++;
+
+       switch (tok) {
+       case (MDOC_Bd):
+               /* FALLTHROUGH */
+       case (MDOC_Bf):
+               /* FALLTHROUGH */
+       case (MDOC_Bl):
+               /* FALLTHROUGH */
+       case (MDOC_Rs):
+               p->norm = mandoc_calloc(1, sizeof(union mdoc_data));
+               break;
+       default:
+               break;
+       }
+
        if ( ! node_append(m, p))
                return(0);
        m->next = MDOC_NEXT_CHILD;
@@ -477,12 +531,37 @@ mdoc_elem_alloc(struct mdoc *m, int line, int pos,
        p->args = args;
        if (p->args)
                (args->refcnt)++;
+
+       switch (tok) {
+       case (MDOC_An):
+               p->norm = mandoc_calloc(1, sizeof(union mdoc_data));
+               break;
+       default:
+               break;
+       }
+
        if ( ! node_append(m, p))
                return(0);
        m->next = MDOC_NEXT_CHILD;
        return(1);
 }
 
+static int
+mdoc_span_alloc(struct mdoc *m, const struct tbl_span *sp)
+{
+       struct mdoc_node *n;
+
+       /* FIXME: grab from tbl_span. */
+       n = node_alloc(m, 0, 0, MDOC_MAX, MDOC_TBL);
+       n->span = sp;
+
+       if ( ! node_append(m, n))
+               return(0);
+
+       m->next = MDOC_NEXT_SIBLING;
+       return(1);
+}
+
 
 int
 mdoc_word_alloc(struct mdoc *m, int line, int pos, const char *p)
@@ -511,25 +590,8 @@ static void
 mdoc_node_free(struct mdoc_node *p)
 {
 
-       /*
-        * XXX: if these end up being problematic in terms of memory
-        * management and dereferencing freed blocks, then make them
-        * into reference-counted double-pointers.
-        */
-
-       if (MDOC_Bd == p->tok && MDOC_BLOCK == p->type)
-               if (p->data.Bd)
-                       free(p->data.Bd);
-       if (MDOC_Bl == p->tok && MDOC_BLOCK == p->type)
-               if (p->data.Bl)
-                       free(p->data.Bl);
-       if (MDOC_Bf == p->tok && MDOC_HEAD == p->type)
-               if (p->data.Bf)
-                       free(p->data.Bf);
-       if (MDOC_An == p->tok)
-               if (p->data.An)
-                       free(p->data.An);
-
+       if (MDOC_BLOCK == p->type || MDOC_ELEM == p->type)
+               free(p->norm);
        if (p->string)
                free(p->string);
        if (p->args)
@@ -605,13 +667,17 @@ mdoc_ptext(struct mdoc *m, int line, char *buf, int offs)
 
        if ('\\' == buf[offs] && 
                        '.' == buf[offs + 1] && 
-                       '"' == buf[offs + 2])
-               return(mdoc_pmsg(m, line, offs, MANDOCERR_BADCOMMENT));
+                       '"' == buf[offs + 2]) {
+               mdoc_pmsg(m, line, offs, MANDOCERR_BADCOMMENT);
+               return(1);
+       }
 
        /* No text before an initial macro. */
 
-       if (SEC_NONE == m->lastnamed)
-               return(mdoc_pmsg(m, line, offs, MANDOCERR_NOTEXT));
+       if (SEC_NONE == m->lastnamed) {
+               mdoc_pmsg(m, line, offs, MANDOCERR_NOTEXT);
+               return(1);
+       }
 
        assert(m->last);
        n = m->last;
@@ -624,7 +690,7 @@ mdoc_ptext(struct mdoc *m, int line, char *buf, int offs)
         */
 
        if (MDOC_Bl == n->tok && MDOC_BODY == n->type &&
-                       LIST_column == n->data.Bl->type) {
+                       LIST_column == n->norm->Bl.type) {
                /* `Bl' is open without any children. */
                m->flags |= MDOC_FREECOL;
                return(mdoc_macro(m, MDOC_It, line, offs, &offs, buf));
@@ -633,7 +699,7 @@ mdoc_ptext(struct mdoc *m, int line, char *buf, int offs)
        if (MDOC_It == n->tok && MDOC_BLOCK == n->type &&
                        NULL != n->parent &&
                        MDOC_Bl == n->parent->tok &&
-                       LIST_column == n->parent->data.Bl->type) {
+                       LIST_column == n->parent->norm->Bl.type) {
                /* `Bl' has block-level `It' children. */
                m->flags |= MDOC_FREECOL;
                return(mdoc_macro(m, MDOC_It, line, offs, &offs, buf));
@@ -686,12 +752,10 @@ mdoc_ptext(struct mdoc *m, int line, char *buf, int offs)
        *end = '\0';
 
        if (ws)
-               if ( ! mdoc_pmsg(m, line, (int)(ws-buf), MANDOCERR_EOLNSPACE))
-                       return(0);
+               mdoc_pmsg(m, line, (int)(ws-buf), MANDOCERR_EOLNSPACE);
 
        if ('\0' == buf[offs] && ! (MDOC_LITERAL & m->flags)) {
-               if ( ! mdoc_pmsg(m, line, (int)(c-buf), MANDOCERR_NOBLANKLN))
-                       return(0);
+               mdoc_pmsg(m, line, (int)(c-buf), MANDOCERR_NOBLANKLN);
 
                /*
                 * Insert a `sp' in the case of a blank line.  Technically,
@@ -791,8 +855,7 @@ mdoc_pmacro(struct mdoc *m, int ln, char *buf, int offs)
         */
 
        if ('\0' == buf[i] && ' ' == buf[i - 1])
-               if ( ! mdoc_pmsg(m, ln, i - 1, MANDOCERR_EOLNSPACE))
-                       goto err;
+               mdoc_pmsg(m, ln, i - 1, MANDOCERR_EOLNSPACE);
 
        /*
         * If an initial macro or a list invocation, divert directly
@@ -814,7 +877,7 @@ mdoc_pmacro(struct mdoc *m, int ln, char *buf, int offs)
         */
 
        if (MDOC_Bl == n->tok && MDOC_BODY == n->type &&
-                       LIST_column == n->data.Bl->type) {
+                       LIST_column == n->norm->Bl.type) {
                m->flags |= MDOC_FREECOL;
                if ( ! mdoc_macro(m, MDOC_It, ln, sv, &sv, buf))
                        goto err;
@@ -830,7 +893,7 @@ mdoc_pmacro(struct mdoc *m, int ln, char *buf, int offs)
        if (MDOC_It == n->tok && MDOC_BLOCK == n->type &&
                        NULL != n->parent &&
                        MDOC_Bl == n->parent->tok &&
-                       LIST_column == n->parent->data.Bl->type) {
+                       LIST_column == n->parent->norm->Bl.type) {
                m->flags |= MDOC_FREECOL;
                if ( ! mdoc_macro(m, MDOC_It, ln, sv, &sv, buf)) 
                        goto err;