=================================================================== RCS file: /cvs/mandoc/main.c,v retrieving revision 1.114 retrieving revision 1.118 diff -u -p -r1.114 -r1.118 --- mandoc/main.c 2010/12/01 17:00:05 1.114 +++ mandoc/main.c 2010/12/05 15:55:01 1.118 @@ -1,4 +1,4 @@ -/* $Id: main.c,v 1.114 2010/12/01 17:00:05 kristaps Exp $ */ +/* $Id: main.c,v 1.118 2010/12/05 15:55:01 kristaps Exp $ */ /* * Copyright (c) 2008, 2009, 2010 Kristaps Dzonsons * Copyright (c) 2010 Ingo Schwarze @@ -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", @@ -146,6 +145,7 @@ static const char * const mandocerrs[MANDOCERR_MAX] = "NAME section must come first", "bad Boolean value", "child violates parent syntax", + "displays may not be nested", "bad AT&T symbol", "bad standard", "list type repeated", @@ -169,6 +169,7 @@ static const char * const mandocerrs[MANDOCERR_MAX] = "request scope close w/none open", "scope already open", "scope open on exit", + "uname(3) system call failed", "macro requires line argument(s)", "macro requires body argument(s)", "macro requires argument(s)", @@ -183,7 +184,6 @@ static const char * const mandocerrs[MANDOCERR_MAX] = "generic fatal error", "column syntax is inconsistent", - "displays may not be nested", "unsupported display type", "blocks badly nested", "no such block is open", @@ -194,7 +194,6 @@ static const char * const mandocerrs[MANDOCERR_MAX] = "NOT IMPLEMENTED: .so with absolute path or \"..\"", "no document body", "no document prologue", - "utsname system call failed", "static buffer exhausted", }; @@ -213,6 +212,7 @@ static void version(void) __attribute__((noreturn)) static int woptions(struct curparse *, char *); static const char *progname; +static enum mandoclevel file_status = MANDOCLEVEL_OK; static enum mandoclevel exit_status = MANDOCLEVEL_OK; int @@ -349,7 +349,7 @@ pfile(const char *file, struct curparse *curp) if (-1 == (fd = open(file, O_RDONLY, 0))) { perror(file); - exit_status = MANDOCLEVEL_SYSERR; + file_status = MANDOCLEVEL_SYSERR; return(0); } @@ -367,7 +367,7 @@ pfile(const char *file, struct curparse *curp) if (-1 == close(fd)) perror(file); - return(MANDOCLEVEL_FATAL > exit_status ? 1 : 0); + return(MANDOCLEVEL_FATAL > file_status ? 1 : 0); } @@ -471,6 +471,7 @@ fdesc(struct curparse *curp) curp->mdoc = NULL; curp->man = NULL; + file_status = MANDOCLEVEL_OK; /* Make sure the mandotory roff parser is initialised. */ @@ -483,32 +484,32 @@ fdesc(struct curparse *curp) pdesc(curp); - if (MANDOCLEVEL_FATAL <= exit_status) + if (MANDOCLEVEL_FATAL <= file_status) goto cleanup; /* NOTE a parser may not have been assigned, yet. */ if ( ! (curp->man || curp->mdoc)) { fprintf(stderr, "%s: Not a manual\n", curp->file); - exit_status = MANDOCLEVEL_FATAL; + file_status = MANDOCLEVEL_FATAL; goto cleanup; } /* Clean up the parse routine ASTs. */ if (curp->mdoc && ! mdoc_endparse(curp->mdoc)) { - assert(MANDOCLEVEL_FATAL <= exit_status); + assert(MANDOCLEVEL_FATAL <= file_status); goto cleanup; } if (curp->man && ! man_endparse(curp->man)) { - assert(MANDOCLEVEL_FATAL <= exit_status); + assert(MANDOCLEVEL_FATAL <= file_status); goto cleanup; } assert(curp->roff); if ( ! roff_endparse(curp->roff)) { - assert(MANDOCLEVEL_FATAL <= exit_status); + assert(MANDOCLEVEL_FATAL <= file_status); goto cleanup; } @@ -517,7 +518,7 @@ fdesc(struct curparse *curp) * the requested level, do not produce output. */ - if (MANDOCLEVEL_OK != exit_status && curp->wstop) + if (MANDOCLEVEL_OK != file_status && curp->wstop) goto cleanup; /* If unset, allocate output dev now (if applicable). */ @@ -592,6 +593,9 @@ fdesc(struct curparse *curp) assert(curp->roff); roff_reset(curp->roff); + if (exit_status < file_status) + exit_status = file_status; + return; } @@ -609,7 +613,7 @@ pdesc(struct curparse *curp) */ if ( ! read_whole_file(curp, &blk, &with_mmap)) { - exit_status = MANDOCLEVEL_SYSERR; + file_status = MANDOCLEVEL_SYSERR; return; } @@ -762,7 +766,7 @@ rerun: pos = 0; continue; case (ROFF_ERR): - assert(MANDOCLEVEL_FATAL <= exit_status); + assert(MANDOCLEVEL_FATAL <= file_status); break; case (ROFF_SO): if (pfile(ln.buf + of, curp)) { @@ -798,7 +802,7 @@ rerun: curp->line, ln.buf, of); if ( ! rc) { - assert(MANDOCLEVEL_FATAL <= exit_status); + assert(MANDOCLEVEL_FATAL <= file_status); break; } } @@ -980,8 +984,8 @@ mmsg(enum mandocerr t, void *arg, int ln, int col, con fprintf(stderr, ": %s", msg); fputc('\n', stderr); - if (exit_status < level) - exit_status = level; + if (file_status < level) + file_status = level; return(level < MANDOCLEVEL_FATAL); }