fix the section number in the <title> element for preformatted pages;

[mandoc.git] / roff_validate.c

diff --git a/roff_validate.c b/roff_validate.c
index d4cf6f6bafcf5432ef76315b342d522eecab8106..74eedafb958108116e8725c27ed9b2a59c817c69 100644 (file)
--- a/roff_validate.c
+++ b/roff_validate.c
@@ -1,6 +1,6 @@
-/*     $Id: roff_validate.c,v 1.15 2018/12/16 00:17:02 schwarze Exp $ */
+/* $Id: roff_validate.c,v 1.20 2020/06/22 19:20:40 schwarze Exp $ */
 /*
- * Copyright (c) 2010, 2017, 2018 Ingo Schwarze <schwarze@openbsd.org>
+ * Copyright (c) 2010, 2017, 2018, 2020 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
@@ -14,6 +14,8 @@
  * ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
  * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
  */
+#include "config.h"
+
 #include <sys/types.h>
 
 #include <assert.h>
@@ -30,15 +32,19 @@
 typedef        void    (*roff_valid_fp)(ROFF_VALID_ARGS);
 
 static void      roff_valid_br(ROFF_VALID_ARGS);
+static void      roff_valid_fi(ROFF_VALID_ARGS);
 static void      roff_valid_ft(ROFF_VALID_ARGS);
+static void      roff_valid_nf(ROFF_VALID_ARGS);
 static void      roff_valid_sp(ROFF_VALID_ARGS);
 
 static const roff_valid_fp roff_valids[ROFF_MAX] = {
        roff_valid_br,  /* br */
        NULL,  /* ce */
+       roff_valid_fi,  /* fi */
        roff_valid_ft,  /* ft */
        NULL,  /* ll */
        NULL,  /* mc */
+       roff_valid_nf,  /* nf */
        NULL,  /* po */
        NULL,  /* rj */
        roff_valid_sp,  /* sp */
@@ -63,10 +69,6 @@ roff_valid_br(ROFF_VALID_ARGS)
 {
        struct roff_node        *np;
 
-       if (n->child != NULL)
-               mandoc_msg(MANDOCERR_ARG_SKIP,
-                   n->line, n->pos, "br %s", n->child->string);
-
        if (n->next != NULL && n->next->type == ROFFT_TEXT &&
            *n->next->string == ' ') {
                mandoc_msg(MANDOCERR_PAR_SKIP, n->line, n->pos,
@@ -75,7 +77,7 @@ roff_valid_br(ROFF_VALID_ARGS)
                return;
        }
 
-       if ((np = n->prev) == NULL)
+       if ((np = roff_node_prev(n)) == NULL)
                return;
 
        switch (np->tok) {
@@ -91,6 +93,13 @@ roff_valid_br(ROFF_VALID_ARGS)
        }
 }
 
+static void
+roff_valid_fi(ROFF_VALID_ARGS)
+{
+       if ((n->flags & NODE_NOFILL) == 0)
+               mandoc_msg(MANDOCERR_FI_SKIP, n->line, n->pos, "fi");
+}
+
 static void
 roff_valid_ft(ROFF_VALID_ARGS)
 {
@@ -110,17 +119,19 @@ roff_valid_ft(ROFF_VALID_ARGS)
        roff_node_delete(man, n);
 }
 
+static void
+roff_valid_nf(ROFF_VALID_ARGS)
+{
+       if (n->flags & NODE_NOFILL)
+               mandoc_msg(MANDOCERR_NF_SKIP, n->line, n->pos, "nf");
+}
+
 static void
 roff_valid_sp(ROFF_VALID_ARGS)
 {
        struct roff_node        *np;
 
-       if (n->child != NULL && n->child->next != NULL)
-               mandoc_msg(MANDOCERR_ARG_EXCESS,
-                   n->child->next->line, n->child->next->pos,
-                   "sp ... %s", n->child->next->string);
-
-       if ((np = n->prev) == NULL)
+       if ((np = roff_node_prev(n)) == NULL)
                return;
 
        switch (np->tok) {