diff options
author | Kristaps Dzonsons <kristaps@bsd.lv> | 2011-04-13 09:57:08 +0000 |
---|---|---|
committer | Kristaps Dzonsons <kristaps@bsd.lv> | 2011-04-13 09:57:08 +0000 |
commit | d6a0b76fecae20088bd4d276cdd0eb2f041d59ec (patch) | |
tree | a869615bb1e7d8d3f17eb922da61febc148a0ddb /man_validate.c | |
parent | f34dee17801e8624ff26bc038d094572f4758655 (diff) | |
download | mandoc-d6a0b76fecae20088bd4d276cdd0eb2f041d59ec.tar.gz mandoc-d6a0b76fecae20088bd4d276cdd0eb2f041d59ec.tar.zst mandoc-d6a0b76fecae20088bd4d276cdd0eb2f041d59ec.zip |
Remove the warning for empty bodies of `Sh', `Ss', `SH', and `SS'. This
prompted by a TODO by schwarze@, originally from Gleydson Soares, that
an empty `SS' was raising an error (it hasn't for some time). It makes
sense these shouldn't warn, as omitting their contents doesn't change
anything in the structure of the document (groff and mandoc specifically
account for the whitespace between empty sections).
This doesn't change any manuals, which only refer to the line arguments
(or possibly next-line, in the case of man(7) syntax).
Diffstat (limited to 'man_validate.c')
-rw-r--r-- | man_validate.c | 13 |
1 files changed, 5 insertions, 8 deletions
diff --git a/man_validate.c b/man_validate.c index e744ed3c..e0c882d4 100644 --- a/man_validate.c +++ b/man_validate.c @@ -1,4 +1,4 @@ -/* $Id: man_validate.c,v 1.68 2011/04/09 15:29:40 kristaps Exp $ */ +/* $Id: man_validate.c,v 1.69 2011/04/13 09:57:08 kristaps Exp $ */ /* * Copyright (c) 2008, 2009, 2010, 2011 Kristaps Dzonsons <kristaps@bsd.lv> * Copyright (c) 2010 Ingo Schwarze <schwarze@openbsd.org> @@ -325,14 +325,11 @@ static int check_sec(CHKARGS) { - if (MAN_HEAD == n->type && 0 == n->nchild) { - man_nmsg(m, n, MANDOCERR_SYNTARGCOUNT); - return(0); - } else if (MAN_BODY == n->type && 0 == n->nchild) - mandoc_msg(MANDOCERR_ARGCWARN, m->parse, n->line, - n->pos, "want children (have none)"); + if ( ! (MAN_HEAD == n->type && 0 == n->nchild)) + return(1); - return(1); + man_nmsg(m, n, MANDOCERR_SYNTARGCOUNT); + return(0); } |