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 | 04c844c9b370f59b5c07157eec6ba5a879b30218 (patch) | |
| tree | 77c904bb370cdab856881030834ae9e529643308 /mdoc_validate.c | |
| parent | 68183e3516ca405d803db497157cd8246b9b8bad (diff) | |
| download | mandoc-04c844c9b370f59b5c07157eec6ba5a879b30218.tar.gz mandoc-04c844c9b370f59b5c07157eec6ba5a879b30218.tar.zst mandoc-04c844c9b370f59b5c07157eec6ba5a879b30218.zip | |
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 | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/mdoc_validate.c b/mdoc_validate.c index c0469e88..f4a7b748 100644 --- a/mdoc_validate.c +++ b/mdoc_validate.c @@ -1,4 +1,4 @@ -/* $Id: mdoc_validate.c,v 1.366 2018/12/30 00:49:55 schwarze Exp $ */ +/* $Id: mdoc_validate.c,v 1.367 2018/12/31 04:55:47 schwarze Exp $ */ /* * Copyright (c) 2008-2012 Kristaps Dzonsons <kristaps@bsd.lv> * Copyright (c) 2010-2018 Ingo Schwarze <schwarze@openbsd.org> @@ -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 |