]> git.cameronkatri.com Git - mandoc.git/blobdiff - man.c
The sh(1) "test" builtin on Solaris 10 doesn't have -e,
[mandoc.git] / man.c
diff --git a/man.c b/man.c
index e1a7d94d0c46dab2705e513ba5833361d87c3720..31c094e8d62c6fe7c2a8d42b6fe0bec269612aed 100644 (file)
--- a/man.c
+++ b/man.c
-/*     $Id: man.c,v 1.118 2012/07/14 10:47:07 schwarze Exp $ */
+/*     $Id: man.c,v 1.166 2015/10/22 21:54:23 schwarze Exp $ */
 /*
  * Copyright (c) 2008, 2009, 2010, 2011 Kristaps Dzonsons <kristaps@bsd.lv>
+ * Copyright (c) 2013, 2014, 2015 Ingo Schwarze <schwarze@openbsd.org>
+ * Copyright (c) 2011 Joerg Sonnenberger <joerg@netbsd.org>
  *
  * Permission to use, copy, modify, and distribute this software for any
  * purpose with or without fee is hereby granted, provided that the above
  * copyright notice and this permission notice appear in all copies.
  *
- * THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES
+ * THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHORS DISCLAIM ALL WARRANTIES
  * WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
- * MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
+ * MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHORS BE LIABLE FOR
  * ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
  * WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
  * ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
  * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
  */
-#ifdef HAVE_CONFIG_H
 #include "config.h"
-#endif
 
 #include <sys/types.h>
 
 #include <assert.h>
+#include <ctype.h>
 #include <stdarg.h>
 #include <stdlib.h>
 #include <stdio.h>
 #include <string.h>
 
-#include "man.h"
+#include "mandoc_aux.h"
 #include "mandoc.h"
-#include "libman.h"
+#include "roff.h"
+#include "man.h"
 #include "libmandoc.h"
+#include "roff_int.h"
+#include "libman.h"
 
-const  char *const __man_macronames[MAN_MAX] = {                
+const  char *const __man_macronames[MAN_MAX] = {
        "br",           "TH",           "SH",           "SS",
-       "TP",           "LP",           "PP",           "P",
+       "TP",           "LP",           "PP",           "P",
        "IP",           "HP",           "SM",           "SB",
        "BI",           "IB",           "BR",           "RB",
        "R",            "B",            "I",            "IR",
-       "RI",           "na",           "sp",           "nf",
+       "RI",           "sp",           "nf",
        "fi",           "RE",           "RS",           "DT",
        "UC",           "PD",           "AT",           "in",
-       "ft",           "OP",           "EX",           "EE"
+       "ft",           "OP",           "EX",           "EE",
+       "UR",           "UE",           "ll"
        };
 
 const  char * const *man_macronames = __man_macronames;
 
-static struct man_node *man_node_alloc(struct man *, int, int, 
-                               enum man_type, enum mant);
-static int              man_node_append(struct man *, 
-                               struct man_node *);
-static void             man_node_free(struct man_node *);
-static void             man_node_unlink(struct man *, 
-                               struct man_node *);
-static int              man_ptext(struct man *, int, char *, int);
-static int              man_pmacro(struct man *, int, char *, int);
-static void             man_free1(struct man *);
-static void             man_alloc1(struct man *);
-static int              man_descope(struct man *, int, int);
-
-
-const struct man_node *
-man_node(const struct man *m)
-{
-
-       assert( ! (MAN_HALT & m->flags));
-       return(m->first);
-}
-
-
-const struct man_meta *
-man_meta(const struct man *m)
-{
-
-       assert( ! (MAN_HALT & m->flags));
-       return(&m->meta);
-}
-
-
-void
-man_reset(struct man *man)
-{
-
-       man_free1(man);
-       man_alloc1(man);
-}
-
-
-void
-man_free(struct man *man)
-{
-
-       man_free1(man);
-       free(man);
-}
-
-
-struct man *
-man_alloc(struct roff *roff, struct mparse *parse)
-{
-       struct man      *p;
-
-       p = mandoc_calloc(1, sizeof(struct man));
-
-       man_hash_init();
-       p->parse = parse;
-       p->roff = roff;
-
-       man_alloc1(p);
-       return(p);
-}
-
-
-int
-man_endparse(struct man *m)
-{
-
-       assert( ! (MAN_HALT & m->flags));
-       if (man_macroend(m))
-               return(1);
-       m->flags |= MAN_HALT;
-       return(0);
-}
+static void             man_descope(struct roff_man *, int, int);
+static int              man_ptext(struct roff_man *, int, char *, int);
+static int              man_pmacro(struct roff_man *, int, char *, int);
 
 
 int
-man_parseln(struct man *m, int ln, char *buf, int offs)
+man_parseln(struct roff_man *man, int ln, char *buf, int offs)
 {
 
-       m->flags |= MAN_NEWLINE;
-
-       assert( ! (MAN_HALT & m->flags));
+       if (man->last->type != ROFFT_EQN || ln > man->last->line)
+               man->flags |= MAN_NEWLINE;
 
-       return (roff_getcontrol(m->roff, buf, &offs) ?
-                       man_pmacro(m, ln, buf, offs) : 
-                       man_ptext(m, ln, buf, offs));
+       return roff_getcontrol(man->roff, buf, &offs) ?
+           man_pmacro(man, ln, buf, offs) :
+           man_ptext(man, ln, buf, offs);
 }
 
-
 static void
-man_free1(struct man *man)
-{
-
-       if (man->first)
-               man_node_delete(man, man->first);
-       if (man->meta.title)
-               free(man->meta.title);
-       if (man->meta.source)
-               free(man->meta.source);
-       if (man->meta.date)
-               free(man->meta.date);
-       if (man->meta.vol)
-               free(man->meta.vol);
-       if (man->meta.msec)
-               free(man->meta.msec);
-}
-
-
-static void
-man_alloc1(struct man *m)
-{
-
-       memset(&m->meta, 0, sizeof(struct man_meta));
-       m->flags = 0;
-       m->last = mandoc_calloc(1, sizeof(struct man_node));
-       m->first = m->last;
-       m->last->type = MAN_ROOT;
-       m->last->tok = MAN_MAX;
-       m->next = MAN_NEXT_CHILD;
-}
-
-
-static int
-man_node_append(struct man *man, struct man_node *p)
-{
-
-       assert(man->last);
-       assert(man->first);
-       assert(MAN_ROOT != p->type);
-
-       switch (man->next) {
-       case (MAN_NEXT_SIBLING):
-               man->last->next = p;
-               p->prev = man->last;
-               p->parent = man->last->parent;
-               break;
-       case (MAN_NEXT_CHILD):
-               man->last->child = p;
-               p->parent = man->last;
-               break;
-       default:
-               abort();
-               /* NOTREACHED */
-       }
-       
-       assert(p->parent);
-       p->parent->nchild++;
-
-       if ( ! man_valid_pre(man, p))
-               return(0);
-
-       switch (p->type) {
-       case (MAN_HEAD):
-               assert(MAN_BLOCK == p->parent->type);
-               p->parent->head = p;
-               break;
-       case (MAN_TAIL):
-               assert(MAN_BLOCK == p->parent->type);
-               p->parent->tail = p;
-               break;
-       case (MAN_BODY):
-               assert(MAN_BLOCK == p->parent->type);
-               p->parent->body = p;
-               break;
-       default:
-               break;
-       }
-
-       man->last = p;
-
-       switch (p->type) {
-       case (MAN_TBL):
-               /* FALLTHROUGH */
-       case (MAN_TEXT):
-               if ( ! man_valid_post(man))
-                       return(0);
-               break;
-       default:
-               break;
-       }
-
-       return(1);
-}
-
-
-static struct man_node *
-man_node_alloc(struct man *m, int line, int pos, 
-               enum man_type type, enum mant tok)
-{
-       struct man_node *p;
-
-       p = mandoc_calloc(1, sizeof(struct man_node));
-       p->line = line;
-       p->pos = pos;
-       p->type = type;
-       p->tok = tok;
-
-       if (MAN_NEWLINE & m->flags)
-               p->flags |= MAN_LINE;
-       m->flags &= ~MAN_NEWLINE;
-       return(p);
-}
-
-
-int
-man_elem_alloc(struct man *m, int line, int pos, enum mant tok)
-{
-       struct man_node *p;
-
-       p = man_node_alloc(m, line, pos, MAN_ELEM, tok);
-       if ( ! man_node_append(m, p))
-               return(0);
-       m->next = MAN_NEXT_CHILD;
-       return(1);
-}
-
-
-int
-man_tail_alloc(struct man *m, int line, int pos, enum mant tok)
-{
-       struct man_node *p;
-
-       p = man_node_alloc(m, line, pos, MAN_TAIL, tok);
-       if ( ! man_node_append(m, p))
-               return(0);
-       m->next = MAN_NEXT_CHILD;
-       return(1);
-}
-
-
-int
-man_head_alloc(struct man *m, int line, int pos, enum mant tok)
-{
-       struct man_node *p;
-
-       p = man_node_alloc(m, line, pos, MAN_HEAD, tok);
-       if ( ! man_node_append(m, p))
-               return(0);
-       m->next = MAN_NEXT_CHILD;
-       return(1);
-}
-
-
-int
-man_body_alloc(struct man *m, int line, int pos, enum mant tok)
-{
-       struct man_node *p;
-
-       p = man_node_alloc(m, line, pos, MAN_BODY, tok);
-       if ( ! man_node_append(m, p))
-               return(0);
-       m->next = MAN_NEXT_CHILD;
-       return(1);
-}
-
-
-int
-man_block_alloc(struct man *m, int line, int pos, enum mant tok)
-{
-       struct man_node *p;
-
-       p = man_node_alloc(m, line, pos, MAN_BLOCK, tok);
-       if ( ! man_node_append(m, p))
-               return(0);
-       m->next = MAN_NEXT_CHILD;
-       return(1);
-}
-
-int
-man_word_alloc(struct man *m, int line, int pos, const char *word)
-{
-       struct man_node *n;
-
-       n = man_node_alloc(m, line, pos, MAN_TEXT, MAN_MAX);
-       n->string = roff_strdup(m->roff, word);
-
-       if ( ! man_node_append(m, n))
-               return(0);
-
-       m->next = MAN_NEXT_SIBLING;
-       return(1);
-}
-
-
-/*
- * Free all of the resources held by a node.  This does NOT unlink a
- * node from its context; for that, see man_node_unlink().
- */
-static void
-man_node_free(struct man_node *p)
-{
-
-       if (p->string)
-               free(p->string);
-       free(p);
-}
-
-
-void
-man_node_delete(struct man *m, struct man_node *p)
-{
-
-       while (p->child)
-               man_node_delete(m, p->child);
-
-       man_node_unlink(m, p);
-       man_node_free(p);
-}
-
-int
-man_addeqn(struct man *m, const struct eqn *ep)
-{
-       struct man_node *n;
-
-       assert( ! (MAN_HALT & m->flags));
-
-       n = man_node_alloc(m, ep->ln, ep->pos, MAN_EQN, MAN_MAX);
-       n->eqn = ep;
-
-       if ( ! man_node_append(m, n))
-               return(0);
-
-       m->next = MAN_NEXT_SIBLING;
-       return(man_descope(m, ep->ln, ep->pos));
-}
-
-int
-man_addspan(struct man *m, const struct tbl_span *sp)
-{
-       struct man_node *n;
-
-       assert( ! (MAN_HALT & m->flags));
-
-       n = man_node_alloc(m, sp->line, 0, MAN_TBL, MAN_MAX);
-       n->span = sp;
-
-       if ( ! man_node_append(m, n))
-               return(0);
-
-       m->next = MAN_NEXT_SIBLING;
-       return(man_descope(m, sp->line, 0));
-}
-
-static int
-man_descope(struct man *m, int line, int offs)
+man_descope(struct roff_man *man, int line, int offs)
 {
        /*
         * Co-ordinate what happens with having a next-line scope open:
@@ -400,50 +76,51 @@ man_descope(struct man *m, int line, int offs)
         * out the block scope (also if applicable).
         */
 
-       if (MAN_ELINE & m->flags) {
-               m->flags &= ~MAN_ELINE;
-               if ( ! man_unscope(m, m->last->parent, MANDOCERR_MAX))
-                       return(0);
+       if (man->flags & MAN_ELINE) {
+               man->flags &= ~MAN_ELINE;
+               man_unscope(man, man->last->parent);
        }
-
-       if ( ! (MAN_BLINE & m->flags))
-               return(1);
-       m->flags &= ~MAN_BLINE;
-
-       if ( ! man_unscope(m, m->last->parent, MANDOCERR_MAX))
-               return(0);
-       return(man_body_alloc(m, line, offs, m->last->tok));
+       if ( ! (man->flags & MAN_BLINE))
+               return;
+       man->flags &= ~MAN_BLINE;
+       man_unscope(man, man->last->parent);
+       roff_body_alloc(man, line, offs, man->last->tok);
 }
 
 static int
-man_ptext(struct man *m, int line, char *buf, int offs)
+man_ptext(struct roff_man *man, int line, char *buf, int offs)
 {
        int              i;
 
        /* Literal free-form text whitespace is preserved. */
 
-       if (MAN_LITERAL & m->flags) {
-               if ( ! man_word_alloc(m, line, offs, buf + offs))
-                       return(0);
-               return(man_descope(m, line, offs));
+       if (man->flags & MAN_LITERAL) {
+               roff_word_alloc(man, line, offs, buf + offs);
+               man_descope(man, line, offs);
+               return 1;
        }
 
-       /* Pump blank lines directly into the backend. */
-
-       for (i = offs; ' ' == buf[i]; i++)
+       for (i = offs; buf[i] == ' '; i++)
                /* Skip leading whitespace. */ ;
 
-       if ('\0' == buf[i]) {
+       /*
+        * Blank lines are ignored right after headings
+        * but add a single vertical space elsewhere.
+        */
+
+       if (buf[i] == '\0') {
                /* Allocate a blank entry. */
-               if ( ! man_elem_alloc(m, line, offs, MAN_sp))
-                       return(0);
-               m->next = MAN_NEXT_SIBLING;
-               return(1);
+               if (man->last->tok != MAN_SH &&
+                   man->last->tok != MAN_SS) {
+                       roff_elem_alloc(man, line, offs, MAN_sp);
+                       man->next = ROFF_NEXT_SIBLING;
+               }
+               return 1;
        }
 
-       /* 
+       /*
         * Warn if the last un-escaped character is whitespace. Then
-        * strip away the remaining spaces (tabs stay!).   
+        * strip away the remaining spaces (tabs stay!).
         */
 
        i = (int)strlen(buf);
@@ -451,7 +128,8 @@ man_ptext(struct man *m, int line, char *buf, int offs)
 
        if (' ' == buf[i - 1] || '\t' == buf[i - 1]) {
                if (i > 1 && '\\' != buf[i - 2])
-                       man_pmsg(m, line, i - 1, MANDOCERR_EOLNSPACE);
+                       mandoc_msg(MANDOCERR_SPACE_EOL, man->parse,
+                           line, i - 1, NULL);
 
                for (--i; i && ' ' == buf[i]; i--)
                        /* Spin back to non-space. */ ;
@@ -461,9 +139,7 @@ man_ptext(struct man *m, int line, char *buf, int offs)
 
                buf[i] = '\0';
        }
-
-       if ( ! man_word_alloc(m, line, offs, buf + offs))
-               return(0);
+       roff_word_alloc(man, line, offs, buf + offs);
 
        /*
         * End-of-sentence check.  If the last character is an unescaped
@@ -472,220 +148,222 @@ man_ptext(struct man *m, int line, char *buf, int offs)
         */
 
        assert(i);
-       if (mandoc_eos(buf, (size_t)i, 0))
-               m->last->flags |= MAN_EOS;
+       if (mandoc_eos(buf, (size_t)i))
+               man->last->flags |= MAN_EOS;
 
-       return(man_descope(m, line, offs));
+       man_descope(man, line, offs);
+       return 1;
 }
 
 static int
-man_pmacro(struct man *m, int ln, char *buf, int offs)
+man_pmacro(struct roff_man *man, int ln, char *buf, int offs)
 {
+       struct roff_node *n;
+       const char      *cp;
+       int              tok;
        int              i, ppos;
-       enum mant        tok;
+       int              bline;
        char             mac[5];
-       struct man_node *n;
-
-       if ('"' == buf[offs]) {
-               man_pmsg(m, ln, offs, MANDOCERR_BADCOMMENT);
-               return(1);
-       } else if ('\0' == buf[offs])
-               return(1);
 
        ppos = offs;
 
        /*
         * Copy the first word into a nil-terminated buffer.
-        * Stop copying when a tab, space, or eoln is encountered.
+        * Stop when a space, tab, escape, or eoln is encountered.
         */
 
        i = 0;
-       while (i < 4 && '\0' != buf[offs] && 
-                       ' ' != buf[offs] && '\t' != buf[offs])
+       while (i < 4 && strchr(" \t\\", buf[offs]) == NULL)
                mac[i++] = buf[offs++];
 
        mac[i] = '\0';
 
-       tok = (i > 0 && i < 4) ? man_hash_find(mac) : MAN_MAX;
+       tok = (i > 0 && i < 4) ? man_hash_find(mac) : TOKEN_NONE;
+
+       if (tok == TOKEN_NONE) {
+               mandoc_msg(MANDOCERR_MACRO, man->parse,
+                   ln, ppos, buf + ppos - 1);
+               return 1;
+       }
+
+       /* Skip a leading escape sequence or tab. */
 
-       if (MAN_MAX == tok) {
-               mandoc_vmsg(MANDOCERR_MACRO, m->parse, ln, 
-                               ppos, "%s", buf + ppos - 1);
-               return(1);
+       switch (buf[offs]) {
+       case '\\':
+               cp = buf + offs + 1;
+               mandoc_escape(&cp, NULL, NULL);
+               offs = cp - buf;
+               break;
+       case '\t':
+               offs++;
+               break;
+       default:
+               break;
        }
 
-       /* The macro is sane.  Jump to the next word. */
+       /* Jump to the next non-whitespace word. */
 
-       while (buf[offs] && ' ' == buf[offs])
+       while (buf[offs] && buf[offs] == ' ')
                offs++;
 
-       /* 
+       /*
         * Trailing whitespace.  Note that tabs are allowed to be passed
         * into the parser as "text", so we only warn about spaces here.
         */
 
-       if ('\0' == buf[offs] && ' ' == buf[offs - 1])
-               man_pmsg(m, ln, offs - 1, MANDOCERR_EOLNSPACE);
+       if (buf[offs] == '\0' && buf[offs - 1] == ' ')
+               mandoc_msg(MANDOCERR_SPACE_EOL, man->parse,
+                   ln, offs - 1, NULL);
 
-       /* 
-        * Remove prior ELINE macro, as it's being clobbered by a new
-        * macro.  Note that NSCOPED macros do not close out ELINE
-        * macros---they don't print text---so we let those slip by.
+       /*
+        * Some macros break next-line scopes; otherwise, remember
+        * whether we are in next-line scope for a block head.
         */
 
-       if ( ! (MAN_NSCOPED & man_macros[tok].flags) &&
-                       m->flags & MAN_ELINE) {
-               n = m->last;
-               assert(MAN_TEXT != n->type);
+       man_breakscope(man, tok);
+       bline = man->flags & MAN_BLINE;
 
-               /* Remove repeated NSCOPED macros causing ELINE. */
+       /* Call to handler... */
 
-               if (MAN_NSCOPED & man_macros[n->tok].flags)
-                       n = n->parent;
+       assert(man_macros[tok].fp);
+       (*man_macros[tok].fp)(man, tok, ln, ppos, &offs, buf);
 
-               mandoc_vmsg(MANDOCERR_LINESCOPE, m->parse, n->line, 
-                   n->pos, "%s breaks %s", man_macronames[tok],
-                   man_macronames[n->tok]);
+       /* In quick mode (for mandocdb), abort after the NAME section. */
 
-               man_node_delete(m, n);
-               m->flags &= ~MAN_ELINE;
+       if (man->quick && tok == MAN_SH) {
+               n = man->last;
+               if (n->type == ROFFT_BODY &&
+                   strcmp(n->prev->child->string, "NAME"))
+                       return 2;
        }
 
        /*
-        * Remove prior BLINE macro that is being clobbered.
+        * If we are in a next-line scope for a block head,
+        * close it out now and switch to the body,
+        * unless the next-line scope is allowed to continue.
         */
-       if ((m->flags & MAN_BLINE) &&
-           (MAN_BSCOPE & man_macros[tok].flags)) {
-               n = m->last;
-
-               /* Might be a text node like 8 in
-                * .TP 8
-                * .SH foo
-                */
-               if (MAN_TEXT == n->type)
-                       n = n->parent;
 
-               /* Remove element that didn't end BLINE, if any. */
-               if ( ! (MAN_BSCOPE & man_macros[n->tok].flags))
-                       n = n->parent;
+       if ( ! bline || man->flags & MAN_ELINE ||
+           man_macros[tok].flags & MAN_NSCOPED)
+               return 1;
 
-               assert(MAN_HEAD == n->type);
-               n = n->parent;
-               assert(MAN_BLOCK == n->type);
-               assert(MAN_SCOPED & man_macros[n->tok].flags);
+       assert(man->flags & MAN_BLINE);
+       man->flags &= ~MAN_BLINE;
 
-               mandoc_vmsg(MANDOCERR_LINESCOPE, m->parse, n->line, 
-                   n->pos, "%s breaks %s", man_macronames[tok],
-                   man_macronames[n->tok]);
+       man_unscope(man, man->last->parent);
+       roff_body_alloc(man, ln, ppos, man->last->tok);
+       return 1;
+}
 
-               man_node_delete(m, n);
-               m->flags &= ~MAN_BLINE;
-       }
+void
+man_breakscope(struct roff_man *man, int tok)
+{
+       struct roff_node *n;
 
        /*
-        * Save the fact that we're in the next-line for a block.  In
-        * this way, embedded roff instructions can "remember" state
-        * when they exit.
+        * An element next line scope is open,
+        * and the new macro is not allowed inside elements.
+        * Delete the element that is being broken.
         */
 
-       if (MAN_BLINE & m->flags)
-               m->flags |= MAN_BPLINE;
-
-       /* Call to handler... */
-
-       assert(man_macros[tok].fp);
-       if ( ! (*man_macros[tok].fp)(m, tok, ln, ppos, &offs, buf))
-               goto err;
+       if (man->flags & MAN_ELINE && (tok == TOKEN_NONE ||
+           ! (man_macros[tok].flags & MAN_NSCOPED))) {
+               n = man->last;
+               assert(n->type != ROFFT_TEXT);
+               if (man_macros[n->tok].flags & MAN_NSCOPED)
+                       n = n->parent;
 
-       /* 
-        * We weren't in a block-line scope when entering the
-        * above-parsed macro, so return.
-        */
+               mandoc_vmsg(MANDOCERR_BLK_LINE, man->parse,
+                   n->line, n->pos, "%s breaks %s",
+                   tok == TOKEN_NONE ? "TS" : man_macronames[tok],
+                   man_macronames[n->tok]);
 
-       if ( ! (MAN_BPLINE & m->flags)) {
-               m->flags &= ~MAN_ILINE; 
-               return(1);
+               roff_node_delete(man, n);
+               man->flags &= ~MAN_ELINE;
        }
-       m->flags &= ~MAN_BPLINE;
 
        /*
-        * If we're in a block scope, then allow this macro to slip by
-        * without closing scope around it.
+        * Weird special case:
+        * Switching fill mode closes section headers.
         */
 
-       if (MAN_ILINE & m->flags) {
-               m->flags &= ~MAN_ILINE;
-               return(1);
+       if (man->flags & MAN_BLINE &&
+           (tok == MAN_nf || tok == MAN_fi) &&
+           (man->last->tok == MAN_SH || man->last->tok == MAN_SS)) {
+               n = man->last;
+               man_unscope(man, n);
+               roff_body_alloc(man, n->line, n->pos, n->tok);
+               man->flags &= ~MAN_BLINE;
        }
 
-       /* 
-        * If we've opened a new next-line element scope, then return
-        * now, as the next line will close out the block scope.
+       /*
+        * A block header next line scope is open,
+        * and the new macro is not allowed inside block headers.
+        * Delete the block that is being broken.
         */
 
-       if (MAN_ELINE & m->flags)
-               return(1);
-
-       /* Close out the block scope opened in the prior line.  */
-
-       assert(MAN_BLINE & m->flags);
-       m->flags &= ~MAN_BLINE;
+       if (man->flags & MAN_BLINE && (tok == TOKEN_NONE ||
+           man_macros[tok].flags & MAN_BSCOPE)) {
+               n = man->last;
+               if (n->type == ROFFT_TEXT)
+                       n = n->parent;
+               if ( ! (man_macros[n->tok].flags & MAN_BSCOPE))
+                       n = n->parent;
 
-       if ( ! man_unscope(m, m->last->parent, MANDOCERR_MAX))
-               return(0);
-       return(man_body_alloc(m, ln, ppos, m->last->tok));
+               assert(n->type == ROFFT_HEAD);
+               n = n->parent;
+               assert(n->type == ROFFT_BLOCK);
+               assert(man_macros[n->tok].flags & MAN_SCOPED);
 
-err:   /* Error out. */
+               mandoc_vmsg(MANDOCERR_BLK_LINE, man->parse,
+                   n->line, n->pos, "%s breaks %s",
+                   tok == TOKEN_NONE ? "TS" : man_macronames[tok],
+                   man_macronames[n->tok]);
 
-       m->flags |= MAN_HALT;
-       return(0);
+               roff_node_delete(man, n);
+               man->flags &= ~MAN_BLINE;
+       }
 }
 
-/*
- * Unlink a node from its context.  If "m" is provided, the last parse
- * point will also be adjusted accordingly.
- */
-static void
-man_node_unlink(struct man *m, struct man_node *n)
+const struct mparse *
+man_mparse(const struct roff_man *man)
 {
 
-       /* Adjust siblings. */
-
-       if (n->prev)
-               n->prev->next = n->next;
-       if (n->next)
-               n->next->prev = n->prev;
-
-       /* Adjust parent. */
+       assert(man && man->parse);
+       return man->parse;
+}
 
-       if (n->parent) {
-               n->parent->nchild--;
-               if (n->parent->child == n)
-                       n->parent->child = n->prev ? n->prev : n->next;
-       }
+void
+man_state(struct roff_man *man, struct roff_node *n)
+{
 
-       /* Adjust parse point, if applicable. */
-
-       if (m && m->last == n) {
-               /*XXX: this can occur when bailing from validation. */
-               /*assert(NULL == n->next);*/
-               if (n->prev) {
-                       m->last = n->prev;
-                       m->next = MAN_NEXT_SIBLING;
-               } else {
-                       m->last = n->parent;
-                       m->next = MAN_NEXT_CHILD;
-               }
+       switch(n->tok) {
+       case MAN_nf:
+       case MAN_EX:
+               if (man->flags & MAN_LITERAL && ! (n->flags & MAN_VALID))
+                       mandoc_msg(MANDOCERR_NF_SKIP, man->parse,
+                           n->line, n->pos, "nf");
+               man->flags |= MAN_LITERAL;
+               break;
+       case MAN_fi:
+       case MAN_EE:
+               if ( ! (man->flags & MAN_LITERAL) &&
+                    ! (n->flags & MAN_VALID))
+                       mandoc_msg(MANDOCERR_FI_SKIP, man->parse,
+                           n->line, n->pos, "fi");
+               man->flags &= ~MAN_LITERAL;
+               break;
+       default:
+               break;
        }
-
-       if (m && m->first == n)
-               m->first = NULL;
+       man->last->flags |= MAN_VALID;
 }
 
-const struct mparse *
-man_mparse(const struct man *m)
+void
+man_validate(struct roff_man *man)
 {
 
-       assert(m && m->parse);
-       return(m->parse);
+       man->last = man->first;
+       man_node_validate(man);
+       man->flags &= ~MAN_LITERAL;
 }