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.

1 files changed, 1 insertions, 10 deletions

diff --git a/mdoc.c b/mdoc.c
index c70b4838..22101188 100644
--- a/mdoc.c
+++ b/mdoc.c
@@ -1,4 +1,4 @@
-/*	$Id: mdoc.c,v 1.272 2018/12/30 00:49:55 schwarze Exp $ */
+/*	$Id: mdoc.c,v 1.273 2018/12/31 04:55:46 schwarze Exp $ */
 /*
  * Copyright (c) 2008, 2009, 2010, 2011 Kristaps Dzonsons <kristaps@bsd.lv>
  * Copyright (c) 2010, 2012-2018 Ingo Schwarze <schwarze@openbsd.org>
@@ -428,12 +428,3 @@ mdoc_isdelim(const char *p)
 
 	return DELIM_NONE;
 }
-
-void
-mdoc_validate(struct roff_man *mdoc)
-{
-
-	mdoc->last = mdoc->meta.first;
-	mdoc_node_validate(mdoc);
-	mdoc_state_reset(mdoc);
-}