diff options
| author | Ingo Schwarze <schwarze@openbsd.org> | 2018-12-14 05:18:02 +0000 |
|---|---|---|
| committer | Ingo Schwarze <schwarze@openbsd.org> | 2018-12-14 05:18:02 +0000 |
| commit | e3863b1460cd9598ecb57705c9b22c2f2c552953 (patch) | |
| tree | f839c62a271e1da0a9086b6adf22cb9ca99eb7e6 /roff_validate.c | |
| parent | 62de322fe3ae3638668739b266e3332e94b5acb3 (diff) | |
| download | mandoc-e3863b1460cd9598ecb57705c9b22c2f2c552953.tar.gz | |
Almost mechanical diff to remove the "struct mparse *" argument
from mandoc_msg(), where it is no longer used.
While here, rename mandoc_vmsg() to mandoc_msg() and retire the
old version: There is really no point in having another function
merely to save "%s" in a few places.
Minus 140 lines of code.
Diffstat (limited to 'roff_validate.c')
| -rw-r--r-- | roff_validate.c | 15 |
1 files changed, 7 insertions, 8 deletions
diff --git a/roff_validate.c b/roff_validate.c index 54172391..ee14188b 100644 --- a/roff_validate.c +++ b/roff_validate.c @@ -64,12 +64,12 @@ roff_valid_br(ROFF_VALID_ARGS) struct roff_node *np; if (n->child != NULL) - mandoc_vmsg(MANDOCERR_ARG_SKIP, man->parse, + mandoc_msg(MANDOCERR_ARG_SKIP, n->line, n->pos, "br %s", n->child->string); if (n->next != NULL && n->next->type == ROFFT_TEXT && *n->next->string == ' ') { - mandoc_msg(MANDOCERR_PAR_SKIP, man->parse, n->line, n->pos, + mandoc_msg(MANDOCERR_PAR_SKIP, n->line, n->pos, "br before text line with leading blank"); roff_node_delete(man, n); return; @@ -82,7 +82,7 @@ roff_valid_br(ROFF_VALID_ARGS) case ROFF_br: case ROFF_sp: case MDOC_Pp: - mandoc_vmsg(MANDOCERR_PAR_SKIP, man->parse, + mandoc_msg(MANDOCERR_PAR_SKIP, n->line, n->pos, "br after %s", roff_name[np->tok]); roff_node_delete(man, n); break; @@ -128,8 +128,7 @@ roff_valid_ft(ROFF_VALID_ARGS) break; } - mandoc_vmsg(MANDOCERR_FT_BAD, man->parse, - n->line, n->pos, "ft %s", cp); + mandoc_msg(MANDOCERR_FT_BAD, n->line, n->pos, "ft %s", cp); roff_node_delete(man, n); } @@ -139,7 +138,7 @@ roff_valid_sp(ROFF_VALID_ARGS) struct roff_node *np; if (n->child != NULL && n->child->next != NULL) - mandoc_vmsg(MANDOCERR_ARG_EXCESS, man->parse, + mandoc_msg(MANDOCERR_ARG_EXCESS, n->child->next->line, n->child->next->pos, "sp ... %s", n->child->next->string); @@ -148,12 +147,12 @@ roff_valid_sp(ROFF_VALID_ARGS) switch (np->tok) { case ROFF_br: - mandoc_msg(MANDOCERR_PAR_SKIP, man->parse, + mandoc_msg(MANDOCERR_PAR_SKIP, np->line, np->pos, "br before sp"); roff_node_delete(man, np); break; case MDOC_Pp: - mandoc_msg(MANDOCERR_PAR_SKIP, man->parse, + mandoc_msg(MANDOCERR_PAR_SKIP, n->line, n->pos, "sp after Pp"); roff_node_delete(man, n); break; |