Grrr, patch(1) merged the new function to the wrong place in this file.

[mandoc.git] / mdoc_macro.c

diff --git a/mdoc_macro.c b/mdoc_macro.c
index a4a4da012243aef861766a4cd9c52169255c7131..f09d8b29a739a389587373c527dab01ae3e7272b 100644 (file)
--- a/mdoc_macro.c
+++ b/mdoc_macro.c
@@ -1,7 +1,7 @@
-/*     $Id: mdoc_macro.c,v 1.135 2014/07/02 20:19:11 schwarze Exp $ */
+/*     $Id: mdoc_macro.c,v 1.143 2014/09/07 00:05:28 schwarze Exp $ */
 /*
  * Copyright (c) 2008-2012 Kristaps Dzonsons <kristaps@bsd.lv>
- * Copyright (c) 2010, 2012, 2013 Ingo Schwarze <schwarze@openbsd.org>
+ * Copyright (c) 2010, 2012, 2013, 2014 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
@@ -15,9 +15,9 @@
  * 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>
@@ -210,7 +210,7 @@ const       struct mdoc_macro __mdoc_macros[MDOC_MAX] = {
        { in_line_eoln, 0 }, /* sp */
        { in_line_eoln, 0 }, /* %U */
        { phrase_ta, MDOC_CALLABLE | MDOC_PARSED | MDOC_JOIN }, /* Ta */
-       { in_line_eoln, 0 }, /* ll */
+       { in_line_eoln, MDOC_PROLOGUE }, /* ll */
 };
 
 const  struct mdoc_macro * const mdoc_macros = __mdoc_macros;
@@ -234,7 +234,8 @@ mdoc_macroend(struct mdoc *mdoc)
        for ( ; n; n = n->parent)
                if (MDOC_BLOCK == n->type &&
                    MDOC_EXPLICIT & mdoc_macros[n->tok].flags)
-                       mdoc_nmsg(mdoc, n, MANDOCERR_SCOPEEXIT);
+                       mandoc_msg(MANDOCERR_BLK_NOEND, mdoc->parse,
+                           n->line, n->pos, mdoc_macronames[n->tok]);
 
        /* Rewind to the first. */
 
@@ -423,6 +424,8 @@ rew_dohalt(enum mdoct tok, enum mdoc_type type,
                        return(REWIND_NONE);
                /* FALLTHROUGH */
        case MDOC_Sh:
+               if (MDOC_ROOT == p->parent->type)
+                       return(REWIND_THIS);
                if (MDOC_Nd == p->tok || MDOC_Ss == p->tok ||
                    MDOC_Sh == p->tok)
                        return(REWIND_MORE);
@@ -528,7 +531,7 @@ make_pending(struct mdoc_node *broken, enum mdoct tok,
                        taker->pending = broken->pending;
                }
                broken->pending = breaker;
-               mandoc_vmsg(MANDOCERR_BLOCK_NEST, mdoc->parse, line, ppos,
+               mandoc_vmsg(MANDOCERR_BLK_NEST, mdoc->parse, line, ppos,
                    "%s breaks %s", mdoc_macronames[tok],
                    mdoc_macronames[broken->tok]);
                return(1);
@@ -558,7 +561,7 @@ rew_sub(enum mdoc_type t, struct mdoc *mdoc,
                             ! (MDOC_EXPLICIT & mdoc_macros[tok].flags));
                        break;
                case REWIND_FORCE:
-                       mandoc_vmsg(MANDOCERR_SCOPEBROKEN, mdoc->parse,
+                       mandoc_vmsg(MANDOCERR_BLK_BROKEN, mdoc->parse,
                            line, ppos, "%s breaks %s",
                            mdoc_macronames[tok],
                            mdoc_macronames[n->tok]);
@@ -574,7 +577,9 @@ rew_sub(enum mdoc_type t, struct mdoc *mdoc,
                                return(1);
                        /* FALLTHROUGH */
                case REWIND_ERROR:
-                       mdoc_pmsg(mdoc, line, ppos, MANDOCERR_NOSCOPE);
+                       mandoc_msg(MANDOCERR_BLK_NOTOPEN,
+                           mdoc->parse, line, ppos,
+                           mdoc_macronames[tok]);
                        return(1);
                }
                break;
@@ -769,11 +774,12 @@ blk_exp_close(MACRO_PROT_ARGS)
                        later = n;
        }
 
-       if ( ! (MDOC_CALLABLE & mdoc_macros[tok].flags)) {
-               /* FIXME: do this in validate */
-               if (buf[*pos])
-                       mdoc_pmsg(mdoc, line, ppos, MANDOCERR_ARGSLOST);
-
+       if ( ! (MDOC_PARSED & mdoc_macros[tok].flags)) {
+               if ('\0' != buf[*pos])
+                       mandoc_vmsg(MANDOCERR_ARG_SKIP,
+                           mdoc->parse, line, ppos,
+                           "%s %s", mdoc_macronames[tok],
+                           buf + *pos);
                if ( ! rew_sub(MDOC_BODY, mdoc, tok, line, ppos))
                        return(0);
                return(rew_sub(MDOC_BLOCK, mdoc, tok, line, ppos));
@@ -837,7 +843,7 @@ blk_exp_close(MACRO_PROT_ARGS)
 static int
 in_line(MACRO_PROT_ARGS)
 {
-       int              la, scope, cnt, nc, nl;
+       int              la, scope, cnt, mayopen, nc, nl;
        enum margverr    av;
        enum mdoct       ntok;
        enum margserr    ac;
@@ -888,6 +894,7 @@ in_line(MACRO_PROT_ARGS)
                return(0);
        }
 
+       mayopen = 1;
        for (cnt = scope = 0;; ) {
                la = *pos;
                ac = mdoc_args(mdoc, line, pos, buf, tok, &p);
@@ -944,19 +951,19 @@ in_line(MACRO_PROT_ARGS)
                         * If we encounter closing punctuation, no word
                         * has been omitted, no scope is open, and we're
                         * allowed to have an empty element, then start
-                        * a new scope.  `Ar', `Fl', and `Li', only do
-                        * this once per invocation.  There may be more
-                        * of these (all of them?).
+                        * a new scope.
                         */
-                       if (0 == cnt && (nc || MDOC_Li == tok) &&
-                           DELIM_CLOSE == d && ! scope) {
+                       if ((d == DELIM_CLOSE ||
+                            (d == DELIM_MIDDLE && tok == MDOC_Fl)) &&
+                           (nc || tok == MDOC_Li) &&
+                           !scope && !cnt && mayopen) {
                                if ( ! mdoc_elem_alloc(mdoc,
                                    line, ppos, tok, arg))
                                        return(0);
-                               if (MDOC_Ar == tok || MDOC_Li == tok ||
-                                   MDOC_Fl == tok)
-                                       cnt++;
                                scope = 1;
+                               cnt++;
+                               if (MDOC_Li == tok || MDOC_Nm == tok)
+                                       mayopen = 0;
                        }
                        /*
                         * Close out our scope, if one is open, before
@@ -965,14 +972,12 @@ in_line(MACRO_PROT_ARGS)
                        if (scope && ! rew_elem(mdoc, tok))
                                return(0);
                        scope = 0;
-               } else if ( ! scope) {
+               } else if (mayopen && !scope) {
                        if ( ! mdoc_elem_alloc(mdoc, line, ppos, tok, arg))
                                return(0);
                        scope = 1;
-               }
-
-               if (DELIM_NONE == d)
                        cnt++;
+               }
 
                if ( ! dword(mdoc, line, la, p, d,
                    MDOC_JOIN & mdoc_macros[tok].flags))
@@ -1031,6 +1036,23 @@ blk_full(MACRO_PROT_ARGS)
 
        nl = MDOC_NEWLINE & mdoc->flags;
 
+       /* Skip items outside lists. */
+
+       if (tok == MDOC_It) {
+               for (n = mdoc->last; n; n = n->parent)
+                       if (n->tok == MDOC_Bl &&
+                           ! (n->flags & MDOC_VALID))
+                               break;
+               if (n == NULL) {
+                       mandoc_vmsg(MANDOCERR_IT_STRAY, mdoc->parse,
+                           line, ppos, "It %s", buf + *pos);
+                       if ( ! mdoc_elem_alloc(mdoc, line, ppos,
+                           MDOC_br, NULL))
+                               return(0);
+                       return(rew_elem(mdoc, MDOC_br));
+               }
+       }
+
        /* Close out prior implicit scope. */
 
        if ( ! (MDOC_EXPLICIT & mdoc_macros[tok].flags)) {
@@ -1762,7 +1784,8 @@ phrase_ta(MACRO_PROT_ARGS)
        while (NULL != n && MDOC_Bl != n->tok)
                n = n->parent;
        if (NULL == n || LIST_column != n->norm->Bl.type) {
-               mdoc_pmsg(mdoc, line, ppos, MANDOCERR_STRAYTA);
+               mandoc_msg(MANDOCERR_TA_STRAY, mdoc->parse,
+                   line, ppos, "Ta");
                return(1);
        }