diff options
author | Ingo Schwarze <schwarze@openbsd.org> | 2018-12-31 04:55:46 +0000 |
---|---|---|
committer | Ingo Schwarze <schwarze@openbsd.org> | 2018-12-31 04:55:46 +0000 |
commit | c0cea6817b9d23d8e5e4a752fc1dff8dfb604cdc (patch) | |
tree | 8583999131b10c7f020825880396b50813e04620 /mdoc_validate.c | |
parent | 0cd2663fe9e76ba0ad357a566dc3416eccab70e5 (diff) | |
download | mandoc-c0cea6817b9d23d8e5e4a752fc1dff8dfb604cdc.tar.gz |
Cleanup, minus 15 LOC, no functional change:
Simplify the way the man(7) and mdoc(7) validators are called.
Reset the parser state with a common function before calling them.
There is no need to again reset the parser state afterwards,
the parsers are no longer used after validation.
This allows getting rid of man_node_validate() and mdoc_node_validate()
as separate functions.
Diffstat (limited to 'mdoc_validate.c')
-rw-r--r-- | mdoc_validate.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/mdoc_validate.c b/mdoc_validate.c index 29a4da6c..58755a2e 100644 --- a/mdoc_validate.c +++ b/mdoc_validate.c @@ -288,7 +288,7 @@ static const char * const secnames[SEC__MAX] = { /* Validate the subtree rooted at mdoc->last. */ void -mdoc_node_validate(struct roff_man *mdoc) +mdoc_validate(struct roff_man *mdoc) { struct roff_node *n, *np; const v_post *p; @@ -319,7 +319,7 @@ mdoc_node_validate(struct roff_man *mdoc) mdoc->last = mdoc->last->child; while (mdoc->last != NULL) { - mdoc_node_validate(mdoc); + mdoc_validate(mdoc); if (mdoc->last == n) mdoc->last = mdoc->last->child; else |