summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorIngo Schwarze <schwarze@openbsd.org>2014-07-02 13:10:45 +0000
committerIngo Schwarze <schwarze@openbsd.org>2014-07-02 13:10:45 +0000
commite98d951b402f5f61bf0fafc86ae5ad5a03f1c85e (patch)
tree23bbf173d3bf3034ab37c0c4935226c4dc6559c1
parent5974c463fa58c6b119a1cacf57644ad24c66106e (diff)
downloadmandoc-e98d951b402f5f61bf0fafc86ae5ad5a03f1c85e.tar.gz
Disentangle the MANDOCERR_CHILD message, which reported three
completely different things, into three distinct messages. Also mention the macro names we are talking about.
-rw-r--r--mandoc.h4
-rw-r--r--mdoc_validate.c10
-rw-r--r--read.c4
3 files changed, 13 insertions, 5 deletions
diff --git a/mandoc.h b/mandoc.h
index 4b39e30b..e837920f 100644
--- a/mandoc.h
+++ b/mandoc.h
@@ -76,8 +76,9 @@ enum mandocerr {
MANDOCERR_PAR_MOVE, /* moving paragraph macro out of list: macro */
MANDOCERR_NS_SKIP, /* skipping no-space macro */
MANDOCERR_BLOCK_NEST, /* blocks badly nested: macro ... */
- MANDOCERR_CHILD, /* child violates parent syntax */
MANDOCERR_BD_NEST, /* nested displays are not portable: macro ... */
+ MANDOCERR_BL_MOVE, /* moving content out of list: macro */
+ MANDOCERR_VT_CHILD, /* .Vt block has child macro: macro */
MANDOCERR_FI_SKIP, /* fill mode already enabled, skipping .fi */
MANDOCERR_NF_SKIP, /* fill mode already disabled, skipping .nf */
MANDOCERR_LINESCOPE, /* line scope broken: macro breaks macro */
@@ -136,6 +137,7 @@ enum mandocerr {
MANDOCERR_MACRO, /* skipping unknown macro */
MANDOCERR_REQUEST, /* NOT IMPLEMENTED: skipping request */
MANDOCERR_ARGCOUNT, /* argument count wrong */
+ MANDOCERR_RS_SKIP, /* skipping invalid content in .Rs block: macro */
MANDOCERR_STRAYTA, /* skipping column outside column list */
MANDOCERR_NOSCOPE, /* skipping end of block that is not open */
MANDOCERR_SCOPEBROKEN, /* missing end of block */
diff --git a/mdoc_validate.c b/mdoc_validate.c
index cf00a0f4..cf75c67a 100644
--- a/mdoc_validate.c
+++ b/mdoc_validate.c
@@ -1117,7 +1117,8 @@ post_vt(POST_ARGS)
for (n = mdoc->last->child; n; n = n->next)
if (MDOC_TEXT != n->type)
- mdoc_nmsg(mdoc, n, MANDOCERR_CHILD);
+ mandoc_msg(MANDOCERR_VT_CHILD, mdoc->parse,
+ n->line, n->pos, mdoc_macronames[n->tok]);
return(1);
}
@@ -1613,7 +1614,9 @@ post_bl(POST_ARGS)
continue;
}
- mdoc_nmsg(mdoc, nchild, MANDOCERR_CHILD);
+ mandoc_msg(MANDOCERR_BL_MOVE, mdoc->parse,
+ nchild->line, nchild->pos,
+ mdoc_macronames[nchild->tok]);
/*
* Move the node out of the Bl block.
@@ -1782,7 +1785,8 @@ post_rs(POST_ARGS)
}
next = nn->next;
- mdoc_nmsg(mdoc, nn, MANDOCERR_CHILD);
+ mandoc_msg(MANDOCERR_RS_SKIP, mdoc->parse,
+ nn->line, nn->pos, mdoc_macronames[nn->tok]);
mdoc_node_delete(mdoc, nn);
}
diff --git a/read.c b/read.c
index ffb650f3..7908ab22 100644
--- a/read.c
+++ b/read.c
@@ -120,8 +120,9 @@ static const char * const mandocerrs[MANDOCERR_MAX] = {
"moving paragraph macro out of list",
"skipping no-space macro",
"blocks badly nested",
- "child violates parent syntax",
"nested displays are not portable",
+ "moving content out of list",
+ ".Vt block has child macro",
"fill mode already enabled, skipping .fi",
"fill mode already disabled, skipping .nf",
"line scope broken",
@@ -180,6 +181,7 @@ static const char * const mandocerrs[MANDOCERR_MAX] = {
"skipping unknown macro",
"NOT IMPLEMENTED, please use groff: skipping request",
"argument count wrong",
+ "skipping invalid content in .Rs block",
"skipping column outside column list",
"skipping end of block that is not open",
"missing end of block",