]> git.cameronkatri.com Git - mandoc.git/blobdiff - validate.c
Added Fd section validation.
[mandoc.git] / validate.c
index 361fecb6aa719da03ad5ecbbf547a86af99738cd..d5495d4fd66e7eb8577ed4e4606dfd55c2ae784c 100644 (file)
@@ -1,4 +1,4 @@
-/* $Id: validate.c,v 1.54 2009/02/24 11:43:13 kristaps Exp $ */
+/* $Id: validate.c,v 1.57 2009/02/24 13:57:17 kristaps Exp $ */
 /*
  * Copyright (c) 2008 Kristaps Dzonsons <kristaps@kth.se>
  *
 /*
  * Copyright (c) 2008 Kristaps Dzonsons <kristaps@kth.se>
  *
@@ -17,6 +17,7 @@
  * PERFORMANCE OF THIS SOFTWARE.
  */
 #include <assert.h>
  * PERFORMANCE OF THIS SOFTWARE.
  */
 #include <assert.h>
+#include <ctype.h>
 #include <stdlib.h>
 
 #include "private.h"
 #include <stdlib.h>
 
 #include "private.h"
  * the BLOCK case, this is the HEAD, BODY, TAIL and so on.
  */
 
  * the BLOCK case, this is the HEAD, BODY, TAIL and so on.
  */
 
-typedef        int     (*v_pre)(struct mdoc *, struct mdoc_node *);
-typedef        int     (*v_post)(struct mdoc *);
+#define        PRE_ARGS        struct mdoc *mdoc, const struct mdoc_node *n
+#define        POST_ARGS       struct mdoc *mdoc
+
+typedef        int     (*v_pre)(PRE_ARGS);
+typedef        int     (*v_post)(POST_ARGS);
 
 /* FIXME: some sections should only occur in specific msecs. */
 /* FIXME: ignoring Pp. */
 /* FIXME: math symbols. */
 
 /* FIXME: some sections should only occur in specific msecs. */
 /* FIXME: ignoring Pp. */
 /* FIXME: math symbols. */
-/* FIXME: .Fd only in synopsis section. */
 
 struct valids {
        v_pre   *pre;
 
 struct valids {
        v_pre   *pre;
@@ -44,11 +47,13 @@ struct      valids {
 
 /* Utility checks. */
 
 
 /* Utility checks. */
 
-static int     check_parent(struct mdoc *, struct mdoc_node *, 
-                       int, enum mdoc_type);
-static int     check_msec(struct mdoc *, struct mdoc_node *, 
-                       int, enum mdoc_msec *);
-static int     check_stdarg(struct mdoc *, struct mdoc_node *);
+static int     check_parent(PRE_ARGS, int, enum mdoc_type);
+static int     check_msec(PRE_ARGS, int, enum mdoc_msec *);
+static int     check_stdarg(PRE_ARGS);
+
+static int     check_text(struct mdoc *, 
+                       size_t, size_t, const char *);
+
 static int     err_child_lt(struct mdoc *, const char *, int);
 static int     err_child_gt(struct mdoc *, const char *, int);
 static int     warn_child_gt(struct mdoc *, const char *, int);
 static int     err_child_lt(struct mdoc *, const char *, int);
 static int     err_child_gt(struct mdoc *, const char *, int);
 static int     warn_child_gt(struct mdoc *, const char *, int);
@@ -65,50 +70,51 @@ static      inline int err_count(struct mdoc *, const char *,
 
 /* Specific pre-child-parse routines. */
 
 
 /* Specific pre-child-parse routines. */
 
-static int     pre_display(struct mdoc *, struct mdoc_node *);
-static int     pre_sh(struct mdoc *, struct mdoc_node *);
-static int     pre_ss(struct mdoc *, struct mdoc_node *);
-static int     pre_bd(struct mdoc *, struct mdoc_node *);
-static int     pre_bl(struct mdoc *, struct mdoc_node *);
-static int     pre_it(struct mdoc *, struct mdoc_node *);
-static int     pre_cd(struct mdoc *, struct mdoc_node *);
-static int     pre_er(struct mdoc *, struct mdoc_node *);
-static int     pre_ex(struct mdoc *, struct mdoc_node *);
-static int     pre_rv(struct mdoc *, struct mdoc_node *);
-static int     pre_an(struct mdoc *, struct mdoc_node *);
-static int     pre_st(struct mdoc *, struct mdoc_node *);
-static int     pre_prologue(struct mdoc *, struct mdoc_node *);
-static int     pre_prologue(struct mdoc *, struct mdoc_node *);
-static int     pre_prologue(struct mdoc *, struct mdoc_node *);
+static int     pre_display(PRE_ARGS);
+static int     pre_sh(PRE_ARGS);
+static int     pre_ss(PRE_ARGS);
+static int     pre_bd(PRE_ARGS);
+static int     pre_bl(PRE_ARGS);
+static int     pre_it(PRE_ARGS);
+static int     pre_cd(PRE_ARGS);
+static int     pre_er(PRE_ARGS);
+static int     pre_ex(PRE_ARGS);
+static int     pre_rv(PRE_ARGS);
+static int     pre_an(PRE_ARGS);
+static int     pre_st(PRE_ARGS);
+static int     pre_prologue(PRE_ARGS);
+static int     pre_prologue(PRE_ARGS);
+static int     pre_prologue(PRE_ARGS);
 
 /* Specific post-child-parse routines. */
 
 
 /* Specific post-child-parse routines. */
 
-static int     herr_ge1(struct mdoc *);
-static int     herr_le1(struct mdoc *);
-static int     herr_eq0(struct mdoc *);
-static int     eerr_eq0(struct mdoc *);
-static int     eerr_le1(struct mdoc *);
-static int     eerr_le2(struct mdoc *);
-static int     eerr_eq1(struct mdoc *);
-static int     eerr_ge1(struct mdoc *);
-static int     ewarn_eq0(struct mdoc *);
-static int     ewarn_eq1(struct mdoc *);
-static int     bwarn_ge1(struct mdoc *);
-static int     ewarn_ge1(struct mdoc *);
-static int     ebool(struct mdoc *);
-static int     post_sh(struct mdoc *);
-static int     post_sh_body(struct mdoc *);
-static int     post_sh_head(struct mdoc *);
-static int     post_bl(struct mdoc *);
-static int     post_it(struct mdoc *);
-static int     post_ex(struct mdoc *);
-static int     post_an(struct mdoc *);
-static int     post_at(struct mdoc *);
-static int     post_xr(struct mdoc *);
-static int     post_nm(struct mdoc *);
-static int     post_bf(struct mdoc *);
-static int     post_root(struct mdoc *);
-static int     pre_text(struct mdoc *, const struct mdoc_node *);
+static int     herr_ge1(POST_ARGS);
+static int     herr_le1(POST_ARGS);
+static int     herr_eq0(POST_ARGS);
+static int     eerr_eq0(POST_ARGS);
+static int     eerr_le1(POST_ARGS);
+static int     eerr_le2(POST_ARGS);
+static int     eerr_eq1(POST_ARGS);
+static int     eerr_ge1(POST_ARGS);
+static int     ewarn_eq0(POST_ARGS);
+static int     ewarn_eq1(POST_ARGS);
+static int     bwarn_ge1(POST_ARGS);
+static int     ewarn_ge1(POST_ARGS);
+static int     ebool(POST_ARGS);
+
+static int     post_sh(POST_ARGS);
+static int     post_sh_body(POST_ARGS);
+static int     post_sh_head(POST_ARGS);
+static int     post_fd(POST_ARGS);
+static int     post_bl(POST_ARGS);
+static int     post_it(POST_ARGS);
+static int     post_ex(POST_ARGS);
+static int     post_an(POST_ARGS);
+static int     post_at(POST_ARGS);
+static int     post_xr(POST_ARGS);
+static int     post_nm(POST_ARGS);
+static int     post_bf(POST_ARGS);
+static int     post_root(POST_ARGS);
 
 /* Collections of pre-child-parse routines. */
 
 
 /* Collections of pre-child-parse routines. */
 
@@ -150,6 +156,7 @@ static      v_post  posts_bf[] = { herr_le1, post_bf, NULL };
 static v_post  posts_rs[] = { herr_eq0, bwarn_ge1, NULL };
 static v_post  posts_fo[] = { bwarn_ge1, NULL };
 static v_post  posts_bk[] = { herr_eq0, bwarn_ge1, NULL };
 static v_post  posts_rs[] = { herr_eq0, bwarn_ge1, NULL };
 static v_post  posts_fo[] = { bwarn_ge1, NULL };
 static v_post  posts_bk[] = { herr_eq0, bwarn_ge1, NULL };
+static v_post  posts_fd[] = { ewarn_ge1, post_fd, NULL };
 
 /* Per-macro pre- and post-child-check routine collections. */
 
 
 /* Per-macro pre- and post-child-check routine collections. */
 
@@ -178,7 +185,7 @@ const       struct valids mdoc_valids[MDOC_MAX] = {
        { NULL, posts_text },                   /* Ev */ 
        { pres_ex, posts_ex },                  /* Ex */ 
        { NULL, posts_text },                   /* Fa */ 
        { NULL, posts_text },                   /* Ev */ 
        { pres_ex, posts_ex },                  /* Ex */ 
        { NULL, posts_text },                   /* Fa */ 
-       { NULL, posts_wtext },                  /* Fd */
+       { NULL, posts_fd },                     /* Fd */
        { NULL, NULL },                         /* Fl */
        { NULL, posts_text },                   /* Fn */ 
        { NULL, posts_wtext },                  /* Ft */ 
        { NULL, NULL },                         /* Fl */
        { NULL, posts_text },                   /* Fn */ 
        { NULL, posts_wtext },                  /* Ft */ 
@@ -263,13 +270,93 @@ const     struct valids mdoc_valids[MDOC_MAX] = {
 };
 
 
 };
 
 
+int
+mdoc_valid_pre(struct mdoc *mdoc, 
+               const struct mdoc_node *node)
+{
+       v_pre           *p;
+       struct mdoc_arg *argv;
+       size_t           argc, i, j, line, pos;
+       const char      *tp;
+
+       if (MDOC_TEXT == node->type) {
+               tp = node->data.text.string;
+               line = node->line;
+               pos = node->pos;
+               return(check_text(mdoc, line, pos, tp));
+       }
+
+       if (MDOC_BLOCK == node->type || MDOC_ELEM == node->type) {
+               argv = MDOC_BLOCK == node->type ?
+                       node->data.block.argv :
+                       node->data.elem.argv;
+               argc = MDOC_BLOCK == node->type ?
+                       node->data.block.argc :
+                       node->data.elem.argc;
+
+               for (i = 0; i < argc; i++) {
+                       if (0 == argv[i].sz)
+                               continue;
+                       for (j = 0; j < argv[i].sz; j++) {
+                               tp = argv[i].value[j];
+                               line = argv[i].line;
+                               pos = argv[i].pos;
+                               if ( ! check_text(mdoc, line, pos, tp))
+                                       return(0);
+                       }
+               }
+       }
+
+       if (NULL == mdoc_valids[node->tok].pre)
+               return(1);
+       for (p = mdoc_valids[node->tok].pre; *p; p++)
+               if ( ! (*p)(mdoc, node)) 
+                       return(0);
+       return(1);
+}
+
+
+int
+mdoc_valid_post(struct mdoc *mdoc)
+{
+       v_post          *p;
+
+       /*
+        * This check occurs after the macro's children have been filled
+        * in: postfix validation.  Since this happens when we're
+        * rewinding the scope tree, it's possible to have multiple
+        * invocations (as by design, for now), we set bit MDOC_VALID to
+        * indicate that we've validated.
+        */
+
+       if (MDOC_VALID & mdoc->last->flags)
+               return(1);
+       mdoc->last->flags |= MDOC_VALID;
+
+       if (MDOC_TEXT == mdoc->last->type)
+               return(1);
+       if (MDOC_ROOT == mdoc->last->type)
+               return(post_root(mdoc));
+
+       if (NULL == mdoc_valids[mdoc->last->tok].post)
+               return(1);
+       for (p = mdoc_valids[mdoc->last->tok].post; *p; p++)
+               if ( ! (*p)(mdoc)) 
+                       return(0);
+
+       return(1);
+}
+
+
+
 static inline int
 warn_count(struct mdoc *m, const char *k, 
                int want, const char *v, int has)
 {
 
 static inline int
 warn_count(struct mdoc *m, const char *k, 
                int want, const char *v, int has)
 {
 
-       return(mdoc_warn(m, WARN_SYNTAX, "suggests %s %d %s "
-                               "(has %d)", v, want, k, has));
+       return(mdoc_warn(m, WARN_SYNTAX, 
+                               "suggests %s %d %s (has %d)", 
+                               v, want, k, has));
 }
 
 
 }
 
 
@@ -314,7 +401,7 @@ lvl##_child_##name(struct mdoc *mdoc, const char *p, int sz)        \
 
 #define CHECK_BODY_DEFN(name, lvl, func, num)                  \
 static int                                                     \
 
 #define CHECK_BODY_DEFN(name, lvl, func, num)                  \
 static int                                                     \
-b##lvl##_##name(struct mdoc *mdoc)                             \
+b##lvl##_##name(POST_ARGS)                                     \
 {                                                              \
        if (MDOC_BODY != mdoc->last->type)                      \
                return(1);                                      \
 {                                                              \
        if (MDOC_BODY != mdoc->last->type)                      \
                return(1);                                      \
@@ -323,7 +410,7 @@ b##lvl##_##name(struct mdoc *mdoc)                          \
 
 #define CHECK_ELEM_DEFN(name, lvl, func, num)                  \
 static int                                                     \
 
 #define CHECK_ELEM_DEFN(name, lvl, func, num)                  \
 static int                                                     \
-e##lvl##_##name(struct mdoc *mdoc)                             \
+e##lvl##_##name(POST_ARGS)                                     \
 {                                                              \
        assert(MDOC_ELEM == mdoc->last->type);                  \
        return(func(mdoc, "line parameters", (num)));           \
 {                                                              \
        assert(MDOC_ELEM == mdoc->last->type);                  \
        return(func(mdoc, "line parameters", (num)));           \
@@ -331,7 +418,7 @@ e##lvl##_##name(struct mdoc *mdoc)                          \
 
 #define CHECK_HEAD_DEFN(name, lvl, func, num)                  \
 static int                                                     \
 
 #define CHECK_HEAD_DEFN(name, lvl, func, num)                  \
 static int                                                     \
-h##lvl##_##name(struct mdoc *mdoc)                             \
+h##lvl##_##name(POST_ARGS)                                     \
 {                                                              \
        if (MDOC_HEAD != mdoc->last->type)                      \
                return(1);                                      \
 {                                                              \
        if (MDOC_HEAD != mdoc->last->type)                      \
                return(1);                                      \
@@ -359,35 +446,58 @@ CHECK_HEAD_DEFN(ge1, err, err_child_gt, 0)        /* herr_ge1() */
 
 
 static int
 
 
 static int
-check_stdarg(struct mdoc *mdoc, struct mdoc_node *node)
+check_stdarg(PRE_ARGS)
 {
 
 {
 
-       if (MDOC_Std == node->data.elem.argv[0].arg && 
-                       1 == node->data.elem.argc)
+       if (MDOC_Std == n->data.elem.argv[0].arg && 
+                       1 == n->data.elem.argc)
                return(1);
 
                return(1);
 
-       return(mdoc_nwarn(mdoc, node, WARN_COMPAT, 
+       return(mdoc_nwarn(mdoc, n, WARN_COMPAT, 
                                "one argument suggested"));
 }
 
 
 static int
                                "one argument suggested"));
 }
 
 
 static int
-check_msec(struct mdoc *mdoc, struct mdoc_node *node, 
-               int sz, enum mdoc_msec *msecs)
+check_msec(PRE_ARGS, int sz, enum mdoc_msec *msecs)
 {
        int              i;
 
        for (i = 0; i < sz; i++)
                if (msecs[i] == mdoc->meta.msec)
                        return(1);
 {
        int              i;
 
        for (i = 0; i < sz; i++)
                if (msecs[i] == mdoc->meta.msec)
                        return(1);
-       return(mdoc_nwarn(mdoc, node, WARN_COMPAT, 
-                               "wrong manual section"));
+       return(mdoc_nwarn(mdoc, n, WARN_COMPAT, 
+                               "invalid manual section"));
+}
+
+
+static int
+check_text(struct mdoc *mdoc, size_t line, size_t pos, const char *p)
+{
+       size_t           c;
+
+       for ( ; *p; p++) {
+               if ( ! isprint(*p) && '\t' != *p)
+                       return(mdoc_perr(mdoc, line, pos,
+                                       "invalid characters"));
+               if ('\\' != *p)
+                       continue;
+               if ((c = mdoc_isescape(p))) {
+                       p += (c - 1);
+                       continue;
+               }
+               return(mdoc_perr(mdoc, line, pos,
+                                       "invalid escape sequence"));
+       }
+
+       return(1);
 }
 
 
 }
 
 
+
+
 static int
 static int
-check_parent(struct mdoc *mdoc, struct mdoc_node *n, 
-               int tok, enum mdoc_type t)
+check_parent(PRE_ARGS, int tok, enum mdoc_type t)
 {
 
        assert(n->parent);
 {
 
        assert(n->parent);
@@ -402,44 +512,44 @@ check_parent(struct mdoc *mdoc, struct mdoc_node *n,
 
 
 static int
 
 
 static int
-pre_display(struct mdoc *mdoc, struct mdoc_node *node)
+pre_display(PRE_ARGS)
 {
 {
-       struct mdoc_node *n;
+       struct mdoc_node *node;
 
        /* Display elements (`Bd', `D1'...) cannot be nested. */
 
 
        /* Display elements (`Bd', `D1'...) cannot be nested. */
 
-       if (MDOC_BLOCK != node->type)
+       if (MDOC_BLOCK != n->type)
                return(1);
 
        /* LINTED */
                return(1);
 
        /* LINTED */
-       for (n = mdoc->last->parent; n; n = n->parent) 
-               if (MDOC_BLOCK == n->type)
-                       if (MDOC_Bd == n->tok)
+       for (node = mdoc->last->parent; node; node = node->parent) 
+               if (MDOC_BLOCK == node->type)
+                       if (MDOC_Bd == node->tok)
                                break;
                                break;
-       if (NULL == n)
+       if (NULL == node)
                return(1);
 
                return(1);
 
-       return(mdoc_nerr(mdoc, node, "displays may not be nested"));
+       return(mdoc_nerr(mdoc, n, "displays may not be nested"));
 }
 
 
 static int
 }
 
 
 static int
-pre_bl(struct mdoc *mdoc, struct mdoc_node *node)
+pre_bl(PRE_ARGS)
 {
        int              type, err, i;
        struct mdoc_arg *argv;
        size_t           argc;
 
 {
        int              type, err, i;
        struct mdoc_arg *argv;
        size_t           argc;
 
-       if (MDOC_BLOCK != node->type)
+       if (MDOC_BLOCK != n->type)
                return(1);
 
                return(1);
 
-       argc = node->data.block.argc; 
+       argc = n->data.block.argc; 
 
        /* Make sure that only one type of list is specified.  */
 
        /* LINTED */
        for (i = 0, type = err = 0; i < (int)argc; i++) {
 
        /* Make sure that only one type of list is specified.  */
 
        /* LINTED */
        for (i = 0, type = err = 0; i < (int)argc; i++) {
-               argv = &node->data.block.argv[i];
+               argv = &n->data.block.argv[i];
 
                switch (argv->arg) {
                case (MDOC_Bullet):
 
                switch (argv->arg) {
                case (MDOC_Bullet):
@@ -479,22 +589,22 @@ pre_bl(struct mdoc *mdoc, struct mdoc_node *node)
 
 
 static int
 
 
 static int
-pre_bd(struct mdoc *mdoc, struct mdoc_node *node)
+pre_bd(PRE_ARGS)
 {
        int              type, err, i;
        struct mdoc_arg *argv;
        size_t           argc;
 
 {
        int              type, err, i;
        struct mdoc_arg *argv;
        size_t           argc;
 
-       if (MDOC_BLOCK != node->type)
+       if (MDOC_BLOCK != n->type)
                return(1);
 
                return(1);
 
-       argc = node->data.block.argc;
+       argc = n->data.block.argc;
 
        /* Make sure that only one type of display is specified.  */
 
        /* LINTED */
        for (i = 0, err = type = 0; ! err && i < (int)argc; i++) {
 
        /* Make sure that only one type of display is specified.  */
 
        /* LINTED */
        for (i = 0, err = type = 0; ! err && i < (int)argc; i++) {
-               argv = &node->data.block.argv[i];
+               argv = &n->data.block.argv[i];
 
                switch (argv->arg) {
                case (MDOC_Ragged):
 
                switch (argv->arg) {
                case (MDOC_Ragged):
@@ -522,123 +632,120 @@ pre_bd(struct mdoc *mdoc, struct mdoc_node *node)
 
 
 static int
 
 
 static int
-pre_ss(struct mdoc *mdoc, struct mdoc_node *node)
+pre_ss(PRE_ARGS)
 {
 
 {
 
-       if (MDOC_BLOCK != node->type)
+       if (MDOC_BLOCK != n->type)
                return(1);
                return(1);
-       return(check_parent(mdoc, node, MDOC_Sh, MDOC_BODY));
+       return(check_parent(mdoc, n, MDOC_Sh, MDOC_BODY));
 }
 
 
 static int
 }
 
 
 static int
-pre_sh(struct mdoc *mdoc, struct mdoc_node *node)
+pre_sh(PRE_ARGS)
 {
 
 {
 
-       if (MDOC_BLOCK != node->type)
+       if (MDOC_BLOCK != n->type)
                return(1);
                return(1);
-       return(check_parent(mdoc, node, -1, MDOC_ROOT));
+       return(check_parent(mdoc, n, -1, MDOC_ROOT));
 }
 
 
 static int
 }
 
 
 static int
-pre_it(struct mdoc *mdoc, struct mdoc_node *node)
+pre_it(PRE_ARGS)
 {
 
        /* TODO: -width attribute must be specified for -tag. */
        /* TODO: children too big for -width? */
 
 {
 
        /* TODO: -width attribute must be specified for -tag. */
        /* TODO: children too big for -width? */
 
-       if (MDOC_BLOCK != node->type)
+       if (MDOC_BLOCK != n->type)
                return(1);
                return(1);
-       return(check_parent(mdoc, node, MDOC_Bl, MDOC_BODY));
+       return(check_parent(mdoc, n, MDOC_Bl, MDOC_BODY));
 }
 
 
 static int
 }
 
 
 static int
-pre_st(struct mdoc *mdoc, struct mdoc_node *node)
+pre_st(PRE_ARGS)
 {
 
 {
 
-       if (1 == node->data.elem.argc)
+       if (1 == n->data.elem.argc)
                return(1);
                return(1);
-       return(mdoc_nerr(mdoc, node, "one argument required"));
+       return(mdoc_nerr(mdoc, n, "one argument required"));
 }
 
 
 static int
 }
 
 
 static int
-pre_an(struct mdoc *mdoc, struct mdoc_node *node)
+pre_an(PRE_ARGS)
 {
 
 {
 
-       if (1 >= node->data.elem.argc)
+       if (1 >= n->data.elem.argc)
                return(1);
                return(1);
-       return(mdoc_nerr(mdoc, node, "one argument allowed"));
+       return(mdoc_nerr(mdoc, n, "one argument allowed"));
 }
 
 
 static int
 }
 
 
 static int
-pre_rv(struct mdoc *mdoc, struct mdoc_node *node)
+pre_rv(PRE_ARGS)
 {
        enum mdoc_msec msecs[] = { MSEC_2, MSEC_3 };
 
 {
        enum mdoc_msec msecs[] = { MSEC_2, MSEC_3 };
 
-       if ( ! check_msec(mdoc, node, 2, msecs))
+       if ( ! check_msec(mdoc, n, 2, msecs))
                return(0);
                return(0);
-       return(check_stdarg(mdoc, node));
+       return(check_stdarg(mdoc, n));
 }
 
 
 static int
 }
 
 
 static int
-pre_ex(struct mdoc *mdoc, struct mdoc_node *node)
+pre_ex(PRE_ARGS)
 {
        enum mdoc_msec msecs[] = { MSEC_1, MSEC_6, MSEC_8 };
 
 {
        enum mdoc_msec msecs[] = { MSEC_1, MSEC_6, MSEC_8 };
 
-       if ( ! check_msec(mdoc, node, 3, msecs))
+       if ( ! check_msec(mdoc, n, 3, msecs))
                return(0);
                return(0);
-       return(check_stdarg(mdoc, node));
+       return(check_stdarg(mdoc, n));
 }
 
 
 static int
 }
 
 
 static int
-pre_er(struct mdoc *mdoc, struct mdoc_node *node)
+pre_er(PRE_ARGS)
 {
        enum mdoc_msec msecs[] = { MSEC_2 };
 
 {
        enum mdoc_msec msecs[] = { MSEC_2 };
 
-       return(check_msec(mdoc, node, 1, msecs));
+       return(check_msec(mdoc, n, 1, msecs));
 }
 
 
 static int
 }
 
 
 static int
-pre_cd(struct mdoc *mdoc, struct mdoc_node *node)
+pre_cd(PRE_ARGS)
 {
        enum mdoc_msec msecs[] = { MSEC_4 };
 
 {
        enum mdoc_msec msecs[] = { MSEC_4 };
 
-       return(check_msec(mdoc, node, 1, msecs));
+       return(check_msec(mdoc, n, 1, msecs));
 }
 
 
 static int
 }
 
 
 static int
-pre_prologue(struct mdoc *mdoc, struct mdoc_node *node)
+pre_prologue(PRE_ARGS)
 {
 
        if (SEC_PROLOGUE != mdoc->lastnamed)
 {
 
        if (SEC_PROLOGUE != mdoc->lastnamed)
-               return(mdoc_nerr(mdoc, node, "prologue only"));
+               return(mdoc_nerr(mdoc, n, "prologue only"));
 
        /* Check for ordering. */
 
 
        /* Check for ordering. */
 
-       switch (node->tok) {
+       switch (n->tok) {
        case (MDOC_Os):
                if (mdoc->meta.title && mdoc->meta.date)
                        break;
        case (MDOC_Os):
                if (mdoc->meta.title && mdoc->meta.date)
                        break;
-               return(mdoc_nerr(mdoc, node, 
-                               "prologue out-of-order"));
+               return(mdoc_nerr(mdoc, n, "prologue out-of-order"));
        case (MDOC_Dt):
                if (NULL == mdoc->meta.title && mdoc->meta.date)
                        break;
        case (MDOC_Dt):
                if (NULL == mdoc->meta.title && mdoc->meta.date)
                        break;
-               return(mdoc_nerr(mdoc, node, 
-                               "prologue out-of-order"));
+               return(mdoc_nerr(mdoc, n, "prologue out-of-order"));
        case (MDOC_Dd):
                if (NULL == mdoc->meta.title && 0 == mdoc->meta.date)
                        break;
        case (MDOC_Dd):
                if (NULL == mdoc->meta.title && 0 == mdoc->meta.date)
                        break;
-               return(mdoc_nerr(mdoc, node, 
-                               "prologue out-of-order"));
+               return(mdoc_nerr(mdoc, n, "prologue out-of-order"));
        default:
                abort();
                /* NOTREACHED */
        default:
                abort();
                /* NOTREACHED */
@@ -646,7 +753,7 @@ pre_prologue(struct mdoc *mdoc, struct mdoc_node *node)
 
        /* Check for repetition. */
 
 
        /* Check for repetition. */
 
-       switch (node->tok) {
+       switch (n->tok) {
        case (MDOC_Os):
                if (NULL == mdoc->meta.os)
                        return(1);
        case (MDOC_Os):
                if (NULL == mdoc->meta.os)
                        return(1);
@@ -664,12 +771,12 @@ pre_prologue(struct mdoc *mdoc, struct mdoc_node *node)
                /* NOTREACHED */
        }
 
                /* NOTREACHED */
        }
 
-       return(mdoc_nerr(mdoc, node, "prologue repetition"));
+       return(mdoc_nerr(mdoc, n, "prologue repetition"));
 }
 
 
 static int
 }
 
 
 static int
-post_bf(struct mdoc *mdoc)
+post_bf(POST_ARGS)
 {
        char             *p;
        struct mdoc_node *head;
 {
        char             *p;
        struct mdoc_node *head;
@@ -703,7 +810,7 @@ post_bf(struct mdoc *mdoc)
 
 
 static int
 
 
 static int
-post_nm(struct mdoc *mdoc)
+post_nm(POST_ARGS)
 {
 
        if (mdoc->last->child)
 {
 
        if (mdoc->last->child)
@@ -715,7 +822,7 @@ post_nm(struct mdoc *mdoc)
 
 
 static int
 
 
 static int
-post_xr(struct mdoc *mdoc)
+post_xr(POST_ARGS)
 {
        struct mdoc_node *n;
 
 {
        struct mdoc_node *n;
 
@@ -728,7 +835,7 @@ post_xr(struct mdoc *mdoc)
 
 
 static int
 
 
 static int
-post_at(struct mdoc *mdoc)
+post_at(POST_ARGS)
 {
 
        if (NULL == mdoc->last->child)
 {
 
        if (NULL == mdoc->last->child)
@@ -740,7 +847,7 @@ post_at(struct mdoc *mdoc)
 
 
 static int
 
 
 static int
-post_an(struct mdoc *mdoc)
+post_an(POST_ARGS)
 {
 
        if (0 != mdoc->last->data.elem.argc) {
 {
 
        if (0 != mdoc->last->data.elem.argc) {
@@ -756,7 +863,7 @@ post_an(struct mdoc *mdoc)
 
 
 static int
 
 
 static int
-post_ex(struct mdoc *mdoc)
+post_ex(POST_ARGS)
 {
 
        if (0 == mdoc->last->data.elem.argc) {
 {
 
        if (0 == mdoc->last->data.elem.argc) {
@@ -776,7 +883,7 @@ post_ex(struct mdoc *mdoc)
 
 
 static int
 
 
 static int
-post_it(struct mdoc *mdoc)
+post_it(POST_ARGS)
 {
        int               type, sv, i;
 #define        TYPE_NONE        (0)
 {
        int               type, sv, i;
 #define        TYPE_NONE        (0)
@@ -893,7 +1000,7 @@ post_it(struct mdoc *mdoc)
 
 
 static int
 
 
 static int
-post_bl(struct mdoc *mdoc)
+post_bl(POST_ARGS)
 {
        struct mdoc_node        *n;
 
 {
        struct mdoc_node        *n;
 
@@ -938,27 +1045,7 @@ ebool(struct mdoc *mdoc)
 
 
 static int
 
 
 static int
-pre_text(struct mdoc *mdoc, const struct mdoc_node *n)
-{
-       size_t           c;
-       const char      *p;
-
-       for (p = n->data.text.string; *p; p++) {
-               if ('\\' != *p)
-                       continue;
-               if ((c = mdoc_isescape(p))) {
-                       p += (c - 1);
-                       continue;
-               }
-               return(mdoc_nerr(mdoc, n, "bad escape sequence"));
-       }
-
-       return(1);
-}
-
-
-static int
-post_root(struct mdoc *mdoc)
+post_root(POST_ARGS)
 {
 
        if (NULL == mdoc->first->child)
 {
 
        if (NULL == mdoc->first->child)
@@ -978,7 +1065,7 @@ post_root(struct mdoc *mdoc)
 
 
 static int
 
 
 static int
-post_sh(struct mdoc *mdoc)
+post_sh(POST_ARGS)
 {
 
        if (MDOC_HEAD == mdoc->last->type)
 {
 
        if (MDOC_HEAD == mdoc->last->type)
@@ -991,7 +1078,7 @@ post_sh(struct mdoc *mdoc)
 
 
 static int
 
 
 static int
-post_sh_body(struct mdoc *mdoc)
+post_sh_body(POST_ARGS)
 {
        struct mdoc_node *n;
 
 {
        struct mdoc_node *n;
 
@@ -1031,7 +1118,7 @@ post_sh_body(struct mdoc *mdoc)
 
 
 static int
 
 
 static int
-post_sh_head(struct mdoc *mdoc)
+post_sh_head(POST_ARGS)
 {
        char              buf[64];
        enum mdoc_sec     sec;
 {
        char              buf[64];
        enum mdoc_sec     sec;
@@ -1059,52 +1146,12 @@ post_sh_head(struct mdoc *mdoc)
 }
 
 
 }
 
 
-int
-mdoc_valid_pre(struct mdoc *mdoc, struct mdoc_node *node)
-{
-       v_pre           *p;
-
-       if (MDOC_TEXT == node->type)
-               return(pre_text(mdoc, node));
-       assert(MDOC_ROOT != node->type);
-
-       if (NULL == mdoc_valids[node->tok].pre)
-               return(1);
-       for (p = mdoc_valids[node->tok].pre; *p; p++)
-               if ( ! (*p)(mdoc, node)) 
-                       return(0);
-       return(1);
-}
-
-
-int
-mdoc_valid_post(struct mdoc *mdoc)
+static int
+post_fd(POST_ARGS)
 {
 {
-       v_post          *p;
-
-       /*
-        * This check occurs after the macro's children have been filled
-        * in: postfix validation.  Since this happens when we're
-        * rewinding the scope tree, it's possible to have multiple
-        * invocations (as by design, for now), we set bit MDOC_VALID to
-        * indicate that we've validated.
-        */
 
 
-       if (MDOC_VALID & mdoc->last->flags)
+       if (SEC_SYNOPSIS == mdoc->last->sec)
                return(1);
                return(1);
-       mdoc->last->flags |= MDOC_VALID;
-
-       if (MDOC_TEXT == mdoc->last->type)
-               return(1);
-       if (MDOC_ROOT == mdoc->last->type)
-               return(post_root(mdoc));
-
-       if (NULL == mdoc_valids[mdoc->last->tok].post)
-               return(1);
-       for (p = mdoc_valids[mdoc->last->tok].post; *p; p++)
-               if ( ! (*p)(mdoc)) 
-                       return(0);
-
-       return(1);
+       return(mdoc_warn(mdoc, WARN_COMPAT, 
+                       "suggested only in section SYNOPSIS"));
 }
 }
-