diff options
| author |   Ingo Schwarze <schwarze@openbsd.org> | 2014-06-20 17:24:00 +0000 |
|---|---|---|
| committer | Ingo Schwarze <schwarze@openbsd.org> | 2014-06-20 17:24:00 +0000 |
| commit | b3b89a43d7165af5fd4fe84f88cdeef6cb5ee1aa (patch) | |
| tree | 66e68d5d37c2a505037e623de69d48b21809cfd2 /mdoc.c | |
| parent | 8358cc737b994d172459eaf3401807c2b0d3811c (diff) | |
| download | mandoc-b3b89a43d7165af5fd4fe84f88cdeef6cb5ee1aa.tar.gz mandoc-b3b89a43d7165af5fd4fe84f88cdeef6cb5ee1aa.tar.zst mandoc-b3b89a43d7165af5fd4fe84f88cdeef6cb5ee1aa.zip | |
Start systematic improvements of error reporting.
So far, this covers all WARNINGs related to the prologue.
1) hierarchical naming of MANDOCERR_* constants
2) mention the macro name in messages where that adds clarity
3) add one missing MANDOCERR_DATE_MISSING msg
4) fix the wording of one message related to the man(7) prologue
Started on the plane back from Ottawa.
Diffstat (limited to 'mdoc.c')
| -rw-r--r-- | mdoc.c | 8 |
1 files changed, 5 insertions, 3 deletions
@@ -1,4 +1,4 @@ -/* $Id: mdoc.c,v 1.214 2014/04/25 14:11:30 schwarze Exp $ */ +/* $Id: mdoc.c,v 1.215 2014/06/20 17:24:00 schwarze Exp $ */ /* * Copyright (c) 2008, 2009, 2010, 2011 Kristaps Dzonsons <kristaps@bsd.lv> * Copyright (c) 2010, 2012, 2013, 2014 Ingo Schwarze <schwarze@openbsd.org> @@ -305,7 +305,8 @@ mdoc_macro(MACRO_PROT_ARGS) if (MDOC_PROLOGUE & mdoc_macros[tok].flags && MDOC_PBODY & mdoc->flags) { - mdoc_pmsg(mdoc, line, ppos, MANDOCERR_BADBODY); + mandoc_vmsg(MANDOCERR_PROLOG_ONLY, mdoc->parse, + line, ppos, "%s", mdoc_macronames[tok]); return(1); } @@ -313,7 +314,8 @@ mdoc_macro(MACRO_PROT_ARGS) if ( ! (MDOC_PROLOGUE & mdoc_macros[tok].flags) && ! (MDOC_PBODY & mdoc->flags)) { - mdoc_pmsg(mdoc, line, ppos, MANDOCERR_BADPROLOG); + mandoc_vmsg(MANDOCERR_PROLOG_BAD, mdoc->parse, + line, ppos, "%s", mdoc_macronames[tok]); if (NULL == mdoc->meta.msec) mdoc->meta.msec = mandoc_strdup("1"); if (NULL == mdoc->meta.title) |