Finally port the OpenBSD regression suite.

[mandoc.git] / mdoc.c

diff --git a/mdoc.c b/mdoc.c
index d0ccaa731663813c1a5c44d2ff86e4d4bd297334..7c3613f4ce4a8d58074344dde14422fd2177b194 100644 (file)
--- a/mdoc.c
+++ b/mdoc.c
@@ -1,7 +1,7 @@
-/*     $Id: mdoc.c,v 1.246 2015/04/18 17:53:21 schwarze Exp $ */
+/*     $Id: mdoc.c,v 1.259 2017/01/28 23:30:08 schwarze Exp $ */

 /*
  * Copyright (c) 2008, 2009, 2010, 2011 Kristaps Dzonsons <kristaps@bsd.lv>
 /*
  * Copyright (c) 2008, 2009, 2010, 2011 Kristaps Dzonsons <kristaps@bsd.lv>

- * Copyright (c) 2010, 2012-2015 Ingo Schwarze <schwarze@openbsd.org>
+ * Copyright (c) 2010, 2012-2017 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
  *
  * Permission to use, copy, modify, and distribute this software for any
  * purpose with or without fee is hereby granted, provided that the above


@@ -32,9 +32,11 @@
 #include "roff.h"
 #include "mdoc.h"
 #include "libmandoc.h"
 #include "roff.h"
 #include "mdoc.h"
 #include "libmandoc.h"

+#include "roff_int.h"

 #include "libmdoc.h"
 
 const  char *const __mdoc_macronames[MDOC_MAX + 1] = {
 #include "libmdoc.h"
 
 const  char *const __mdoc_macronames[MDOC_MAX + 1] = {

+       "text",

        "Ap",           "Dd",           "Dt",           "Os",
        "Sh",           "Ss",           "Pp",           "D1",
        "Dl",           "Bd",           "Ed",           "Bl",
        "Ap",           "Dd",           "Dt",           "Os",
        "Sh",           "Ss",           "Pp",           "D1",
        "Dl",           "Bd",           "Ed",           "Bl",


@@ -65,8 +67,8 @@ const char *const __mdoc_macronames[MDOC_MAX + 1] = {
        "Lk",           "Mt",           "Brq",          "Bro",
        "Brc",          "%C",           "Es",           "En",
        "Dx",           "%Q",           "br",           "sp",
        "Lk",           "Mt",           "Brq",          "Bro",
        "Brc",          "%C",           "Es",           "En",
        "Dx",           "%Q",           "br",           "sp",

-       "%U",           "Ta",           "ll",           "text",
-       };
+       "%U",           "Ta",           "ll",
+};

 
 const  char *const __mdoc_argnames[MDOC_ARG_MAX] = {
        "split",                "nosplit",              "ragged",
 
 const  char *const __mdoc_argnames[MDOC_ARG_MAX] = {
        "split",                "nosplit",              "ragged",


@@ -80,50 +82,13 @@ const       char *const __mdoc_argnames[MDOC_ARG_MAX] = {
        "symbolic",             "nested",               "centered"
        };
 
        "symbolic",             "nested",               "centered"
        };
 

-const  char * const *mdoc_macronames = __mdoc_macronames;
+const  char * const *mdoc_macronames = __mdoc_macronames + 1;

 const  char * const *mdoc_argnames = __mdoc_argnames;
 
 const  char * const *mdoc_argnames = __mdoc_argnames;
 

-static void              mdoc_node_free(struct roff_node *);
-static void              mdoc_node_unlink(struct roff_man *,
-                               struct roff_node *);
-static struct roff_node *node_alloc(struct roff_man *, int, int,
-                               int, enum roff_type);
-static void              node_append(struct roff_man *, struct roff_node *);

 static int               mdoc_ptext(struct roff_man *, int, char *, int);
 static int               mdoc_pmacro(struct roff_man *, int, char *, int);
 
 
 static int               mdoc_ptext(struct roff_man *, int, char *, int);
 static int               mdoc_pmacro(struct roff_man *, int, char *, int);
 
 

-void
-mdoc_endparse(struct roff_man *mdoc)
-{
-
-       mdoc_macroend(mdoc);
-}
-
-void
-mdoc_addeqn(struct roff_man *mdoc, const struct eqn *ep)
-{
-       struct roff_node *n;
-
-       n = node_alloc(mdoc, ep->ln, ep->pos, MDOC_MAX, ROFFT_EQN);
-       n->eqn = ep;
-       if (ep->ln > mdoc->last->line)
-               n->flags |= MDOC_LINE;
-       node_append(mdoc, n);
-       mdoc->next = ROFF_NEXT_SIBLING;
-}
-
-void
-mdoc_addspan(struct roff_man *mdoc, const struct tbl_span *sp)
-{
-       struct roff_node *n;
-
-       n = node_alloc(mdoc, sp->line, 0, MDOC_MAX, ROFFT_TBL);
-       n->span = sp;
-       node_append(mdoc, n);
-       mdoc->next = ROFF_NEXT_SIBLING;
-}
-

 /*
  * Main parse routine.  Parses a single line -- really just hands off to
  * the macro (mdoc_pmacro()) or text parser (mdoc_ptext()).
 /*
  * Main parse routine.  Parses a single line -- really just hands off to
  * the macro (mdoc_pmacro()) or text parser (mdoc_ptext()).


@@ -146,140 +111,17 @@ mdoc_parseln(struct roff_man *mdoc, int ln, char *buf, int offs)
        else
                mdoc->flags &= ~MDOC_SYNOPSIS;
 
        else
                mdoc->flags &= ~MDOC_SYNOPSIS;
 

-       return(roff_getcontrol(mdoc->roff, buf, &offs) ?
+       return roff_getcontrol(mdoc->roff, buf, &offs) ?

            mdoc_pmacro(mdoc, ln, buf, offs) :
            mdoc_pmacro(mdoc, ln, buf, offs) :

-           mdoc_ptext(mdoc, ln, buf, offs));
+           mdoc_ptext(mdoc, ln, buf, offs);

 }
 
 void
 mdoc_macro(MACRO_PROT_ARGS)
 {
 }
 
 void
 mdoc_macro(MACRO_PROT_ARGS)
 {

-       assert(tok < MDOC_MAX);
-
-       if (mdoc->flags & MDOC_PBODY) {
-               if (tok == MDOC_Dt) {
-                       mandoc_vmsg(MANDOCERR_DT_LATE,
-                           mdoc->parse, line, ppos,
-                           "Dt %s", buf + *pos);
-                       return;
-               }
-       } else if ( ! (mdoc_macros[tok].flags & MDOC_PROLOGUE)) {
-               if (mdoc->meta.title == NULL) {
-                       mandoc_vmsg(MANDOCERR_DT_NOTITLE,
-                           mdoc->parse, line, ppos, "%s %s",
-                           mdoc_macronames[tok], buf + *pos);
-                       mdoc->meta.title = mandoc_strdup("UNTITLED");
-               }
-               if (NULL == mdoc->meta.vol)
-                       mdoc->meta.vol = mandoc_strdup("LOCAL");
-               mdoc->flags |= MDOC_PBODY;
-       }
-       (*mdoc_macros[tok].fp)(mdoc, tok, line, ppos, pos, buf);
-}
-
-
-static void
-node_append(struct roff_man *mdoc, struct roff_node *p)
-{
-
-       assert(mdoc->last);
-       assert(mdoc->first);
-       assert(p->type != ROFFT_ROOT);
-
-       switch (mdoc->next) {
-       case ROFF_NEXT_SIBLING:
-               mdoc->last->next = p;
-               p->prev = mdoc->last;
-               p->parent = mdoc->last->parent;
-               break;
-       case ROFF_NEXT_CHILD:
-               mdoc->last->child = p;
-               p->parent = mdoc->last;
-               break;
-       default:
-               abort();
-               /* NOTREACHED */
-       }
-
-       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 ROFFT_BODY:
-               if (ENDBODY_NOT != p->end)
-                       break;
-               /* FALLTHROUGH */
-       case ROFFT_TAIL:
-               /* FALLTHROUGH */
-       case ROFFT_HEAD:
-               p->norm = p->parent->norm;
-               break;
-       default:
-               break;
-       }
-
-       mdoc_valid_pre(mdoc, p);
-
-       switch (p->type) {
-       case ROFFT_HEAD:
-               assert(p->parent->type == ROFFT_BLOCK);
-               p->parent->head = p;
-               break;
-       case ROFFT_TAIL:
-               assert(p->parent->type == ROFFT_BLOCK);
-               p->parent->tail = p;
-               break;
-       case ROFFT_BODY:
-               if (p->end)
-                       break;
-               assert(p->parent->type == ROFFT_BLOCK);
-               p->parent->body = p;
-               break;
-       default:
-               break;
-       }
-
-       mdoc->last = p;
-
-       switch (p->type) {
-       case ROFFT_TBL:
-               /* FALLTHROUGH */
-       case ROFFT_TEXT:
-               mdoc_valid_post(mdoc);
-               break;
-       default:
-               break;
-       }
-}
-
-static struct roff_node *
-node_alloc(struct roff_man *mdoc, int line, int pos,
-       int tok, enum roff_type type)
-{
-       struct roff_node *p;
-
-       p = mandoc_calloc(1, sizeof(*p));
-       p->sec = mdoc->lastsec;
-       p->line = line;
-       p->pos = pos;
-       p->tok = tok;
-       p->type = type;
-
-       /* Flag analysis. */
+       assert(tok > TOKEN_NONE && tok < MDOC_MAX);

 
 

-       if (MDOC_SYNOPSIS & mdoc->flags)
-               p->flags |= MDOC_SYNPRETTY;
-       else
-               p->flags &= ~MDOC_SYNPRETTY;
-       if (MDOC_NEWLINE & mdoc->flags)
-               p->flags |= MDOC_LINE;
-       mdoc->flags &= ~MDOC_NEWLINE;
-
-       return(p);
+       (*mdoc_macros[tok].fp)(mdoc, tok, line, ppos, pos, buf);

 }
 
 void
 }
 
 void


@@ -287,33 +129,9 @@ mdoc_tail_alloc(struct roff_man *mdoc, int line, int pos, int tok)
 {
        struct roff_node *p;
 
 {
        struct roff_node *p;
 

-       p = node_alloc(mdoc, line, pos, tok, ROFFT_TAIL);
-       node_append(mdoc, p);
-       mdoc->next = ROFF_NEXT_CHILD;
-}
-
-struct roff_node *
-mdoc_head_alloc(struct roff_man *mdoc, int line, int pos, int tok)
-{
-       struct roff_node *p;
-
-       assert(mdoc->first);
-       assert(mdoc->last);
-       p = node_alloc(mdoc, line, pos, tok, ROFFT_HEAD);
-       node_append(mdoc, p);
-       mdoc->next = ROFF_NEXT_CHILD;
-       return(p);
-}
-
-struct roff_node *
-mdoc_body_alloc(struct roff_man *mdoc, int line, int pos, int tok)
-{
-       struct roff_node *p;
-
-       p = node_alloc(mdoc, line, pos, tok, ROFFT_BODY);
-       node_append(mdoc, p);
+       p = roff_node_alloc(mdoc, line, pos, ROFFT_TAIL, tok);
+       roff_node_append(mdoc, p);

        mdoc->next = ROFF_NEXT_CHILD;
        mdoc->next = ROFF_NEXT_CHILD;

-       return(p);

 }
 
 struct roff_node *
 }
 
 struct roff_node *


@@ -322,15 +140,15 @@ mdoc_endbody_alloc(struct roff_man *mdoc, int line, int pos, int tok,
 {
        struct roff_node *p;
 
 {
        struct roff_node *p;
 

-       body->flags |= MDOC_ENDED;
-       body->parent->flags |= MDOC_ENDED;
-       p = node_alloc(mdoc, line, pos, tok, ROFFT_BODY);
+       body->flags |= NODE_ENDED;
+       body->parent->flags |= NODE_ENDED;
+       p = roff_node_alloc(mdoc, line, pos, ROFFT_BODY, tok);

        p->body = body;
        p->norm = body->norm;
        p->end = end;
        p->body = body;
        p->norm = body->norm;
        p->end = end;

-       node_append(mdoc, p);
+       roff_node_append(mdoc, p);

        mdoc->next = ROFF_NEXT_SIBLING;
        mdoc->next = ROFF_NEXT_SIBLING;

-       return(p);
+       return p;

 }
 
 struct roff_node *
 }
 
 struct roff_node *


@@ -339,29 +157,25 @@ mdoc_block_alloc(struct roff_man *mdoc, int line, int pos,
 {
        struct roff_node *p;
 
 {
        struct roff_node *p;
 

-       p = node_alloc(mdoc, line, pos, tok, ROFFT_BLOCK);
+       p = roff_node_alloc(mdoc, line, pos, ROFFT_BLOCK, tok);

        p->args = args;
        if (p->args)
                (args->refcnt)++;
 
        switch (tok) {
        case MDOC_Bd:
        p->args = args;
        if (p->args)
                (args->refcnt)++;
 
        switch (tok) {
        case MDOC_Bd:

-               /* FALLTHROUGH */

        case MDOC_Bf:
        case MDOC_Bf:

-               /* FALLTHROUGH */

        case MDOC_Bl:
        case MDOC_Bl:

-               /* FALLTHROUGH */

        case MDOC_En:
        case MDOC_En:

-               /* FALLTHROUGH */

        case MDOC_Rs:
                p->norm = mandoc_calloc(1, sizeof(union mdoc_data));
                break;
        default:
                break;
        }
        case MDOC_Rs:
                p->norm = mandoc_calloc(1, sizeof(union mdoc_data));
                break;
        default:
                break;
        }

-       node_append(mdoc, p);
+       roff_node_append(mdoc, p);

        mdoc->next = ROFF_NEXT_CHILD;
        mdoc->next = ROFF_NEXT_CHILD;

-       return(p);
+       return p;

 }
 
 void
 }
 
 void


@@ -370,7 +184,7 @@ mdoc_elem_alloc(struct roff_man *mdoc, int line, int pos,
 {
        struct roff_node *p;
 
 {
        struct roff_node *p;
 

-       p = node_alloc(mdoc, line, pos, tok, ROFFT_ELEM);
+       p = roff_node_alloc(mdoc, line, pos, ROFFT_ELEM, tok);

        p->args = args;
        if (p->args)
                (args->refcnt)++;
        p->args = args;
        if (p->args)
                (args->refcnt)++;


@@ -382,106 +196,17 @@ mdoc_elem_alloc(struct roff_man *mdoc, int line, int pos,
        default:
                break;
        }
        default:
                break;
        }

-       node_append(mdoc, p);
+       roff_node_append(mdoc, p);

        mdoc->next = ROFF_NEXT_CHILD;
 }
 
        mdoc->next = ROFF_NEXT_CHILD;
 }
 

-void
-mdoc_word_alloc(struct roff_man *mdoc, int line, int pos, const char *p)
-{
-       struct roff_node *n;
-
-       n = node_alloc(mdoc, line, pos, MDOC_MAX, ROFFT_TEXT);
-       n->string = roff_strdup(mdoc->roff, p);
-       node_append(mdoc, n);
-       mdoc->next = ROFF_NEXT_SIBLING;
-}
-
-void
-mdoc_word_append(struct roff_man *mdoc, const char *p)
-{
-       struct roff_node        *n;
-       char                    *addstr, *newstr;
-
-       n = mdoc->last;
-       addstr = roff_strdup(mdoc->roff, p);
-       mandoc_asprintf(&newstr, "%s %s", n->string, addstr);
-       free(addstr);
-       free(n->string);
-       n->string = newstr;
-       mdoc->next = ROFF_NEXT_SIBLING;
-}
-
-static void
-mdoc_node_free(struct roff_node *p)
-{
-
-       if (p->type == ROFFT_BLOCK || p->type == ROFFT_ELEM)
-               free(p->norm);
-       if (p->string)
-               free(p->string);
-       if (p->args)
-               mdoc_argv_free(p->args);
-       free(p);
-}
-
-static void
-mdoc_node_unlink(struct roff_man *mdoc, struct roff_node *n)
-{
-
-       /* Adjust siblings. */
-
-       if (n->prev)
-               n->prev->next = n->next;
-       if (n->next)
-               n->next->prev = n->prev;
-
-       /* Adjust parent. */
-
-       if (n->parent) {
-               n->parent->nchild--;
-               if (n->parent->child == n)
-                       n->parent->child = n->prev ? n->prev : n->next;
-               if (n->parent->last == n)
-                       n->parent->last = n->prev ? n->prev : NULL;
-       }
-
-       /* Adjust parse point, if applicable. */
-
-       if (mdoc && mdoc->last == n) {
-               if (n->prev) {
-                       mdoc->last = n->prev;
-                       mdoc->next = ROFF_NEXT_SIBLING;
-               } else {
-                       mdoc->last = n->parent;
-                       mdoc->next = ROFF_NEXT_CHILD;
-               }
-       }
-
-       if (mdoc && mdoc->first == n)
-               mdoc->first = NULL;
-}
-
-void
-mdoc_node_delete(struct roff_man *mdoc, struct roff_node *p)
-{
-
-       while (p->child) {
-               assert(p->nchild);
-               mdoc_node_delete(mdoc, p->child);
-       }
-       assert(0 == p->nchild);
-
-       mdoc_node_unlink(mdoc, p);
-       mdoc_node_free(p);
-}
-

 void
 mdoc_node_relink(struct roff_man *mdoc, struct roff_node *p)
 {
 
 void
 mdoc_node_relink(struct roff_man *mdoc, struct roff_node *p)
 {
 

-       mdoc_node_unlink(mdoc, p);
-       node_append(mdoc, p);
+       roff_node_unlink(mdoc, p);
+       p->prev = p->next = NULL;
+       roff_node_append(mdoc, p);

 }
 
 /*
 }
 
 /*


@@ -494,32 +219,22 @@ mdoc_ptext(struct roff_man *mdoc, int line, char *buf, int offs)
        struct roff_node *n;
        char             *c, *ws, *end;
 
        struct roff_node *n;
        char             *c, *ws, *end;
 

-       assert(mdoc->last);

        n = mdoc->last;
 
        /*
        n = mdoc->last;
 
        /*

-        * Divert directly to list processing if we're encountering a
-        * columnar ROFFT_BLOCK with or without a prior ROFFT_BLOCK entry
-        * (a ROFFT_BODY means it's already open, in which case we should
-        * process within its context in the normal way).
+        * If a column list contains plain text, assume an implicit item
+        * macro.  This can happen one or more times at the beginning
+        * of such a list, intermixed with non-It mdoc macros and with
+        * nodes generated on the roff level, for example by tbl.

         */
 
         */
 

-       if (n->tok == MDOC_Bl && n->type == ROFFT_BODY &&
-           n->end == ENDBODY_NOT && n->norm->Bl.type == LIST_column) {
-               /* `Bl' is open without any children. */
-               mdoc->flags |= MDOC_FREECOL;
-               mdoc_macro(mdoc, MDOC_It, line, offs, &offs, buf);
-               return(1);
-       }
-
-       if (n->tok == MDOC_It && n->type == ROFFT_BLOCK &&
-           NULL != n->parent &&
-           MDOC_Bl == n->parent->tok &&
-           LIST_column == n->parent->norm->Bl.type) {
-               /* `Bl' has block-level `It' children. */
+       if ((n->tok == MDOC_Bl && n->type == ROFFT_BODY &&
+            n->end == ENDBODY_NOT && n->norm->Bl.type == LIST_column) ||
+           (n->parent != NULL && n->parent->tok == MDOC_Bl &&
+            n->parent->norm->Bl.type == LIST_column)) {

                mdoc->flags |= MDOC_FREECOL;
                mdoc_macro(mdoc, MDOC_It, line, offs, &offs, buf);
                mdoc->flags |= MDOC_FREECOL;
                mdoc_macro(mdoc, MDOC_It, line, offs, &offs, buf);

-               return(1);
+               return 1;

        }
 
        /*
        }
 
        /*


@@ -576,16 +291,16 @@ mdoc_ptext(struct roff_man *mdoc, int line, char *buf, int offs)
                 * blank lines aren't allowed, but enough manuals assume this
                 * behaviour that we want to work around it.
                 */
                 * blank lines aren't allowed, but enough manuals assume this
                 * behaviour that we want to work around it.
                 */

-               mdoc_elem_alloc(mdoc, line, offs, MDOC_sp, NULL);
+               roff_elem_alloc(mdoc, line, offs, MDOC_sp);
+               mdoc->last->flags |= NODE_VALID | NODE_ENDED;

                mdoc->next = ROFF_NEXT_SIBLING;
                mdoc->next = ROFF_NEXT_SIBLING;

-               mdoc_valid_post(mdoc);
-               return(1);
+               return 1;

        }
 
        }
 

-       mdoc_word_alloc(mdoc, line, offs, buf+offs);
+       roff_word_alloc(mdoc, line, offs, buf+offs);

 
        if (mdoc->flags & MDOC_LITERAL)
 
        if (mdoc->flags & MDOC_LITERAL)

-               return(1);
+               return 1;

 
        /*
         * End-of-sentence check.  If the last character is an unescaped
 
        /*
         * End-of-sentence check.  If the last character is an unescaped


@@ -596,8 +311,24 @@ mdoc_ptext(struct roff_man *mdoc, int line, char *buf, int offs)
        assert(buf < end);
 
        if (mandoc_eos(buf+offs, (size_t)(end-buf-offs)))
        assert(buf < end);
 
        if (mandoc_eos(buf+offs, (size_t)(end-buf-offs)))

-               mdoc->last->flags |= MDOC_EOS;
-       return(1);
+               mdoc->last->flags |= NODE_EOS;
+
+       for (c = buf + offs; c != NULL; c = strchr(c + 1, '.')) {
+               if (c - buf < offs + 2)
+                       continue;
+               if (end - c < 4)
+                       break;
+               if (isalpha((unsigned char)c[-2]) &&
+                   isalpha((unsigned char)c[-1]) &&
+                   c[1] == ' ' &&
+                   isupper((unsigned char)(c[2] == ' ' ? c[3] : c[2])) &&
+                   (c[-2] != 'n' || c[-1] != 'c') &&
+                   (c[-2] != 'v' || c[-1] != 's'))
+                       mandoc_msg(MANDOCERR_EOS, mdoc->parse,
+                           line, (int)(c - buf), NULL);
+       }
+
+       return 1;

 }
 
 /*
 }
 
 /*


@@ -626,12 +357,12 @@ mdoc_pmacro(struct roff_man *mdoc, int ln, char *buf, int offs)
 
        mac[i] = '\0';
 
 
        mac[i] = '\0';
 

-       tok = (i > 1 && i < 4) ? mdoc_hash_find(mac) : MDOC_MAX;
+       tok = (i > 1 && i < 4) ? mdoc_hash_find(mac) : TOKEN_NONE;

 
 

-       if (tok == MDOC_MAX) {
+       if (tok == TOKEN_NONE) {

                mandoc_msg(MANDOCERR_MACRO, mdoc->parse,
                    ln, sv, buf + sv - 1);
                mandoc_msg(MANDOCERR_MACRO, mdoc->parse,
                    ln, sv, buf + sv - 1);

-               return(1);
+               return 1;

        }
 
        /* Skip a leading escape sequence or tab. */
        }
 
        /* Skip a leading escape sequence or tab. */


@@ -668,39 +399,26 @@ mdoc_pmacro(struct roff_man *mdoc, int ln, char *buf, int offs)
         * into macro processing.
         */
 
         * into macro processing.
         */
 

-       if (NULL == mdoc->last || MDOC_It == tok || MDOC_El == tok) {
-               mdoc_macro(mdoc, tok, ln, sv, &offs, buf);
-               return(1);
-       }
-

        n = mdoc->last;
        n = mdoc->last;

-       assert(mdoc->last);
-
-       /*
-        * If the first macro of a `Bl -column', open an `It' block
-        * context around the parsed macro.
-        */
-
-       if (n->tok == MDOC_Bl && n->type == ROFFT_BODY &&
-           n->end == ENDBODY_NOT && n->norm->Bl.type == LIST_column) {
-               mdoc->flags |= MDOC_FREECOL;
-               mdoc_macro(mdoc, MDOC_It, ln, sv, &sv, buf);
-               return(1);
+       if (n == NULL || tok == MDOC_It || tok == MDOC_El) {
+               mdoc_macro(mdoc, tok, ln, sv, &offs, buf);
+               return 1;

        }
 
        /*
        }
 
        /*

-        * If we're following a block-level `It' within a `Bl -column'
-        * context (perhaps opened in the above block or in ptext()),
-        * then open an `It' block context around the parsed macro.
+        * If a column list contains a non-It macro, assume an implicit
+        * item macro.  This can happen one or more times at the
+        * beginning of such a list, intermixed with text lines and
+        * with nodes generated on the roff level, for example by tbl.

         */
 
         */
 

-       if (n->tok == MDOC_It && n->type == ROFFT_BLOCK &&
-           NULL != n->parent &&
-           MDOC_Bl == n->parent->tok &&
-           LIST_column == n->parent->norm->Bl.type) {
+       if ((n->tok == MDOC_Bl && n->type == ROFFT_BODY &&
+            n->end == ENDBODY_NOT && n->norm->Bl.type == LIST_column) ||
+           (n->parent != NULL && n->parent->tok == MDOC_Bl &&
+            n->parent->norm->Bl.type == LIST_column)) {

                mdoc->flags |= MDOC_FREECOL;
                mdoc_macro(mdoc, MDOC_It, ln, sv, &sv, buf);
                mdoc->flags |= MDOC_FREECOL;
                mdoc_macro(mdoc, MDOC_It, ln, sv, &sv, buf);

-               return(1);
+               return 1;

        }
 
        /* Normal processing of a macro. */
        }
 
        /* Normal processing of a macro. */


@@ -711,9 +429,9 @@ mdoc_pmacro(struct roff_man *mdoc, int ln, char *buf, int offs)
 
        if (mdoc->quick && MDOC_Sh == tok &&
            SEC_NAME != mdoc->last->sec)
 
        if (mdoc->quick && MDOC_Sh == tok &&
            SEC_NAME != mdoc->last->sec)

-               return(2);
+               return 2;

 
 

-       return(1);
+       return 1;

 }
 
 enum mdelim
 }
 
 enum mdelim


@@ -721,82 +439,44 @@ mdoc_isdelim(const char *p)
 {
 
        if ('\0' == p[0])
 {
 
        if ('\0' == p[0])

-               return(DELIM_NONE);
+               return DELIM_NONE;

 
        if ('\0' == p[1])
                switch (p[0]) {
                case '(':
 
        if ('\0' == p[1])
                switch (p[0]) {
                case '(':

-                       /* FALLTHROUGH */

                case '[':
                case '[':

-                       return(DELIM_OPEN);
+                       return DELIM_OPEN;

                case '|':
                case '|':

-                       return(DELIM_MIDDLE);
+                       return DELIM_MIDDLE;

                case '.':
                case '.':

-                       /* FALLTHROUGH */

                case ',':
                case ',':

-                       /* FALLTHROUGH */

                case ';':
                case ';':

-                       /* FALLTHROUGH */

                case ':':
                case ':':

-                       /* FALLTHROUGH */

                case '?':
                case '?':

-                       /* FALLTHROUGH */

                case '!':
                case '!':

-                       /* FALLTHROUGH */

                case ')':
                case ')':

-                       /* FALLTHROUGH */

                case ']':
                case ']':

-                       return(DELIM_CLOSE);
+                       return DELIM_CLOSE;

                default:
                default:

-                       return(DELIM_NONE);
+                       return DELIM_NONE;

                }
 
        if ('\\' != p[0])
                }
 
        if ('\\' != p[0])

-               return(DELIM_NONE);
+               return DELIM_NONE;

 
        if (0 == strcmp(p + 1, "."))
 
        if (0 == strcmp(p + 1, "."))

-               return(DELIM_CLOSE);
+               return DELIM_CLOSE;

        if (0 == strcmp(p + 1, "fR|\\fP"))
        if (0 == strcmp(p + 1, "fR|\\fP"))

-               return(DELIM_MIDDLE);
+               return DELIM_MIDDLE;

 
 

-       return(DELIM_NONE);
+       return DELIM_NONE;

 }
 
 void
 }
 
 void

-mdoc_deroff(char **dest, const struct roff_node *n)
+mdoc_validate(struct roff_man *mdoc)

 {
 {

-       char    *cp;
-       size_t   sz;
-
-       if (n->type != ROFFT_TEXT) {
-               for (n = n->child; n; n = n->next)
-                       mdoc_deroff(dest, n);
-               return;
-       }
-
-       /* Skip leading whitespace. */
-
-       for (cp = n->string; '\0' != *cp; cp++)
-               if (0 == isspace((unsigned char)*cp))
-                       break;
-
-       /* Skip trailing whitespace. */
-
-       for (sz = strlen(cp); sz; sz--)
-               if (0 == isspace((unsigned char)cp[sz-1]))
-                       break;
-
-       /* Skip empty strings. */
-
-       if (0 == sz)
-               return;
-
-       if (NULL == *dest) {
-               *dest = mandoc_strndup(cp, sz);
-               return;
-       }

 
 

-       mandoc_asprintf(&cp, "%s %*s", *dest, (int)sz, cp);
-       free(*dest);
-       *dest = cp;
+       mdoc->last = mdoc->first;
+       mdoc_node_validate(mdoc);
+       mdoc_state_reset(mdoc);

 }
 }