summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorKristaps Dzonsons <kristaps@bsd.lv>2010-12-05 15:55:01 +0000
committerKristaps Dzonsons <kristaps@bsd.lv>2010-12-05 15:55:01 +0000
commitda864306deafefa98f63fe802fa901c9aec68f65 (patch)
treed2c992b5d14975fd6dd68e2ec32344c1a233d599
parent17d87274a35b068e7bf8fa10374f640943cf7bb4 (diff)
downloadmandoc-da864306deafefa98f63fe802fa901c9aec68f65.tar.gz
Remove long-standing FIXME regarding the consolidation of WIDTHARG and
NOARGV warnings. From patch by schwarze@.
-rw-r--r--main.c1
-rw-r--r--mandoc.h2
-rw-r--r--mdoc_validate.c5
3 files changed, 2 insertions, 6 deletions
diff --git a/main.c b/main.c
index e884a636..64166f6c 100644
--- a/main.c
+++ b/main.c
@@ -133,7 +133,6 @@ static const char * const mandocerrs[MANDOCERR_MAX] = {
"bad escape sequence",
"unterminated quoted string",
"argument requires the width argument",
- "superfluous width argument",
"bad date argument",
"bad width argument",
"unknown manual section",
diff --git a/mandoc.h b/mandoc.h
index de4eedb1..3b3e14d1 100644
--- a/mandoc.h
+++ b/mandoc.h
@@ -56,8 +56,6 @@ enum mandocerr {
MANDOCERR_BADESCAPE, /* bad escape sequence */
MANDOCERR_BADQUOTE, /* unterminated quoted string */
MANDOCERR_NOWIDTHARG, /* argument requires the width argument */
- /* FIXME: merge with MANDOCERR_IGNARGV. */
- MANDOCERR_WIDTHARG, /* superfluous width argument */
MANDOCERR_BADDATE, /* bad date argument */
MANDOCERR_BADWIDTH, /* bad width argument */
MANDOCERR_BADMSEC, /* unknown manual section */
diff --git a/mdoc_validate.c b/mdoc_validate.c
index fa9f3e13..c25bbf6d 100644
--- a/mdoc_validate.c
+++ b/mdoc_validate.c
@@ -753,9 +753,8 @@ pre_bl(PRE_ARGS)
case (LIST_inset):
/* FALLTHROUGH */
case (LIST_item):
- if (NULL == n->data.Bl->width)
- break;
- mdoc_nmsg(mdoc, n, MANDOCERR_WIDTHARG);
+ if (n->data.Bl->width)
+ mdoc_nmsg(mdoc, n, MANDOCERR_IGNARGV);
break;
default:
break;