=================================================================== RCS file: /cvs/mandoc/main.c,v retrieving revision 1.2 retrieving revision 1.53 diff -u -p -r1.2 -r1.53 --- mandoc/main.c 2009/03/19 16:18:36 1.2 +++ mandoc/main.c 2009/10/31 06:10:57 1.53 @@ -1,102 +1,150 @@ -/* $Id: main.c,v 1.2 2009/03/19 16:18:36 kristaps Exp $ */ +/* $Id: main.c,v 1.53 2009/10/31 06:10:57 kristaps Exp $ */ /* - * Copyright (c) 2008, 2009 Kristaps Dzonsons + * Copyright (c) 2008, 2009 Kristaps Dzonsons * * Permission to use, copy, modify, and distribute this software for any - * purpose with or without fee is hereby granted, provided that the - * above copyright notice and this permission notice appear in all - * copies. + * purpose with or without fee is hereby granted, provided that the above + * copyright notice and this permission notice appear in all copies. * - * THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL - * WARRANTIES WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED - * WARRANTIES OF MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE - * AUTHOR BE LIABLE FOR ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL - * DAMAGES OR ANY DAMAGES WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR - * PROFITS, WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE OR OTHER - * TORTIOUS ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR - * PERFORMANCE OF THIS SOFTWARE. + * THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES + * WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF + * MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR + * ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES + * WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN + * ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF + * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE. */ #include #include -#include #include #include +#include #include #include #include #include "mdoc.h" +#include "man.h" +#include "main.h" -#define WARN_WALL 0x03 /* All-warnings mask. */ -#define WARN_WCOMPAT (1 << 0) /* Compatibility warnings. */ -#define WARN_WSYNTAX (1 << 1) /* Syntax warnings. */ -#define WARN_WERR (1 << 2) /* Warnings->errors. */ +#define UNCONST(a) ((void *)(uintptr_t)(const void *)(a)) -enum outt { - OUTT_ASCII, - OUTT_LATIN1, - OUTT_UTF8, - OUTT_TREE, - OUTT_LINT -}; +/* Account for FreeBSD and Linux in our declarations. */ -typedef int (*out_run)(void *, const struct mdoc *); +#ifdef __linux__ +extern int getsubopt(char **, char * const *, char **); +extern size_t strlcat(char *, const char *, size_t); +# ifndef __dead +# define __dead __attribute__((__noreturn__)) +# endif +#elif defined(__dead2) +# ifndef __dead +# define __dead __dead2 +# endif +#endif + +typedef void (*out_mdoc)(void *, const struct mdoc *); +typedef void (*out_man)(void *, const struct man *); typedef void (*out_free)(void *); -extern char *__progname; +struct buf { + char *buf; + size_t sz; +}; -extern void *ascii_alloc(void); -extern void *latin1_alloc(void); -extern void *utf8_alloc(void); -extern int terminal_run(void *, const struct mdoc *); -extern int tree_run(void *, const struct mdoc *); -extern void terminal_free(void *); +enum intt { + INTT_AUTO, + INTT_MDOC, + INTT_MAN +}; -__dead static void version(void); -__dead static void usage(void); +enum outt { + OUTT_ASCII = 0, + OUTT_TREE, + OUTT_HTML, + OUTT_LINT +}; + +struct curparse { + const char *file; /* Current parse. */ + int fd; /* Current parse. */ + int wflags; +#define WARN_WALL (1 << 0) /* All-warnings mask. */ +#define WARN_WERR (1 << 2) /* Warnings->errors. */ + int fflags; +#define IGN_SCOPE (1 << 0) /* Ignore scope errors. */ +#define NO_IGN_ESCAPE (1 << 1) /* Don't ignore bad escapes. */ +#define NO_IGN_MACRO (1 << 2) /* Don't ignore bad macros. */ +#define NO_IGN_CHARS (1 << 3) /* Don't ignore bad chars. */ +#define IGN_ERRORS (1 << 4) /* Ignore failed parse. */ + enum intt inttype; /* Input parsers... */ + struct man *man; + struct man *lastman; + struct mdoc *mdoc; + struct mdoc *lastmdoc; + enum outt outtype; /* Output devices... */ + out_mdoc outmdoc; + out_man outman; + out_free outfree; + void *outdata; + char outopts[BUFSIZ]; +}; + static int foptions(int *, char *); static int toptions(enum outt *, char *); +static int moptions(enum intt *, char *); static int woptions(int *, char *); static int merr(void *, int, int, const char *); -static int mwarn(void *, int, int, - enum mdoc_warn, const char *); -static int file(char **, size_t *, char **, size_t *, - const char *, struct mdoc *); -static int fdesc(char **, size_t *, char **, size_t *, - const char *, int, struct mdoc *); +static int mwarn(void *, int, int, const char *); +static int ffile(struct buf *, struct buf *, + const char *, struct curparse *); +static int fdesc(struct buf *, struct buf *, + struct curparse *); +static int pset(const char *, int, struct curparse *, + struct man **, struct mdoc **); +static struct man *man_init(struct curparse *); +static struct mdoc *mdoc_init(struct curparse *); +__dead static void version(void); +__dead static void usage(void); +extern char *__progname; + int main(int argc, char *argv[]) { - int c, rc, fflags, wflags; - struct mdoc_cb cb; - struct mdoc *mdoc; - char *buf, *line; - size_t bufsz, linesz; - void *outdata; - enum outt outtype; - out_run outrun; - out_free outfree; + int c, rc; + struct buf ln, blk; + struct curparse curp; - fflags = wflags = 0; - outtype = OUTT_ASCII; + memset(&curp, 0, sizeof(struct curparse)); + curp.inttype = INTT_AUTO; + curp.outtype = OUTT_ASCII; + /* LINTED */ - while (-1 != (c = getopt(argc, argv, "f:VW:T:"))) + while (-1 != (c = getopt(argc, argv, "f:m:O:T:VW:"))) switch (c) { case ('f'): - if ( ! foptions(&fflags, optarg)) - return(0); + if ( ! foptions(&curp.fflags, optarg)) + return(EXIT_FAILURE); break; + case ('m'): + if ( ! moptions(&curp.inttype, optarg)) + return(EXIT_FAILURE); + break; + case ('O'): + (void)strlcat(curp.outopts, optarg, BUFSIZ); + (void)strlcat(curp.outopts, ",", BUFSIZ); + break; case ('T'): - if ( ! toptions(&outtype, optarg)) - return(0); + if ( ! toptions(&curp.outtype, optarg)) + return(EXIT_FAILURE); break; case ('W'): - if ( ! woptions(&wflags, optarg)) - return(0); + if ( ! woptions(&curp.wflags, optarg)) + return(EXIT_FAILURE); break; case ('V'): version(); @@ -109,77 +157,51 @@ main(int argc, char *argv[]) argc -= optind; argv += optind; - /* - * Allocate the appropriate front-end. Note that utf8, ascii - * and latin1 all resolve to the terminal front-end with - * different encodings (see terminal.c). Not all frontends have - * cleanup or alloc routines. - */ + memset(&ln, 0, sizeof(struct buf)); + memset(&blk, 0, sizeof(struct buf)); - switch (outtype) { - case (OUTT_LATIN1): - outdata = latin1_alloc(); - outrun = terminal_run; - outfree = terminal_free; - break; - case (OUTT_UTF8): - outdata = utf8_alloc(); - outrun = terminal_run; - outfree = terminal_free; - break; - case (OUTT_TREE): - outdata = NULL; - outrun = tree_run; - outfree = NULL; - break; - case (OUTT_LINT): - outdata = NULL; - outrun = NULL; - outfree = NULL; - break; - default: - outdata = ascii_alloc(); - outrun = terminal_run; - outfree = terminal_free; - break; - } + rc = 1; - /* - * All callbacks route into here, where we print them onto the - * screen. XXX - for now, no path for debugging messages. - */ + if (NULL == *argv) { + curp.file = ""; + curp.fd = STDIN_FILENO; - cb.mdoc_msg = NULL; - cb.mdoc_err = merr; - cb.mdoc_warn = mwarn; + c = fdesc(&blk, &ln, &curp); + if ( ! (IGN_ERRORS & curp.fflags)) + rc = 1 == c ? 1 : 0; + else + rc = -1 == c ? 0 : 1; + } - buf = line = NULL; - bufsz = linesz = 0; + while (rc && *argv) { + c = ffile(&blk, &ln, *argv, &curp); + if ( ! (IGN_ERRORS & curp.fflags)) + rc = 1 == c ? 1 : 0; + else + rc = -1 == c ? 0 : 1; - mdoc = mdoc_alloc(&wflags, fflags, &cb); - - while (*argv) { - if ( ! file(&line, &linesz, &buf, &bufsz, *argv, mdoc)) - break; - if (outrun && ! (*outrun)(outdata, mdoc)) - break; - - /* Reset the parser for another file. */ - mdoc_reset(mdoc); argv++; + if (*argv && rc) { + if (curp.lastman) + man_reset(curp.lastman); + if (curp.lastmdoc) + mdoc_reset(curp.lastmdoc); + curp.lastman = NULL; + curp.lastmdoc = NULL; + } } - rc = NULL == *argv; + if (blk.buf) + free(blk.buf); + if (ln.buf) + free(ln.buf); + if (curp.outfree) + (*curp.outfree)(curp.outdata); + if (curp.mdoc) + mdoc_free(curp.mdoc); + if (curp.man) + man_free(curp.man); - if (buf) - free(buf); - if (line) - free(line); - if (outfree) - (*outfree)(outdata); - - mdoc_free(mdoc); - return(rc ? EXIT_SUCCESS : EXIT_FAILURE); } @@ -189,8 +211,7 @@ version(void) { (void)printf("%s %s\n", __progname, VERSION); - exit(0); - /* NOTREACHED */ + exit(EXIT_SUCCESS); } @@ -198,97 +219,168 @@ __dead static void usage(void) { - (void)fprintf(stderr, "usage: %s\n", __progname); - exit(1); - /* NOTREACHED */ + (void)fprintf(stderr, "usage: %s [-V] [-foption...] " + "[-mformat] [-Ooption] [-Toutput] " + "[-Werr...]\n", __progname); + exit(EXIT_FAILURE); } +static struct man * +man_init(struct curparse *curp) +{ + int pflags; + struct man_cb mancb; + + mancb.man_err = merr; + mancb.man_warn = mwarn; + + /* Defaults from mandoc.1. */ + + pflags = MAN_IGN_MACRO | MAN_IGN_ESCAPE | MAN_IGN_CHARS; + + if (curp->fflags & NO_IGN_MACRO) + pflags &= ~MAN_IGN_MACRO; + if (curp->fflags & NO_IGN_CHARS) + pflags &= ~MAN_IGN_CHARS; + if (curp->fflags & NO_IGN_ESCAPE) + pflags &= ~MAN_IGN_ESCAPE; + + return(man_alloc(curp, pflags, &mancb)); +} + + +static struct mdoc * +mdoc_init(struct curparse *curp) +{ + int pflags; + struct mdoc_cb mdoccb; + + mdoccb.mdoc_err = merr; + mdoccb.mdoc_warn = mwarn; + + /* Defaults from mandoc.1. */ + + pflags = MDOC_IGN_MACRO | MDOC_IGN_ESCAPE | MDOC_IGN_CHARS; + + if (curp->fflags & IGN_SCOPE) + pflags |= MDOC_IGN_SCOPE; + if (curp->fflags & NO_IGN_ESCAPE) + pflags &= ~MDOC_IGN_ESCAPE; + if (curp->fflags & NO_IGN_MACRO) + pflags &= ~MDOC_IGN_MACRO; + if (curp->fflags & NO_IGN_CHARS) + pflags &= ~MDOC_IGN_CHARS; + + return(mdoc_alloc(curp, pflags, &mdoccb)); +} + + static int -file(char **ln, size_t *lnsz, char **buf, size_t *bufsz, - const char *file, struct mdoc *mdoc) +ffile(struct buf *blk, struct buf *ln, + const char *file, struct curparse *curp) { - int fd, c; + int c; - if (-1 == (fd = open(file, O_RDONLY, 0))) { - warn("%s", file); - return(0); + curp->file = file; + if (-1 == (curp->fd = open(curp->file, O_RDONLY, 0))) { + perror(curp->file); + return(-1); } - c = fdesc(ln, lnsz, buf, bufsz, file, fd, mdoc); + c = fdesc(blk, ln, curp); - if (-1 == close(fd)) - warn("%s", file); + if (-1 == close(curp->fd)) + perror(curp->file); return(c); } static int -fdesc(char **lnp, size_t *lnsz, char **bufp, size_t *bufsz, - const char *f, int fd, struct mdoc *mdoc) +fdesc(struct buf *blk, struct buf *ln, struct curparse *curp) { size_t sz; ssize_t ssz; struct stat st; - int j, i, pos, lnn; - char *ln, *buf; + int j, i, pos, lnn, comment; + struct man *man; + struct mdoc *mdoc; - buf = *bufp; - ln = *lnp; + sz = BUFSIZ; + man = NULL; + mdoc = NULL; /* - * Two buffers: ln and buf. buf is the input buffer, optimised - * for each file's block size. ln is a line buffer. Both + * Two buffers: ln and buf. buf is the input buffer optimised + * here for each file's block size. ln is a line buffer. Both * growable, hence passed in by ptr-ptr. */ - if (-1 == fstat(fd, &st)) { - warnx("%s", f); - sz = BUFSIZ; - } else - sz = (unsigned)BUFSIZ > st.st_blksize ? - (size_t)BUFSIZ : st.st_blksize; + if (-1 == fstat(curp->fd, &st)) + perror(curp->file); + else if ((size_t)st.st_blksize > sz) + sz = st.st_blksize; - if (sz > *bufsz) { - if (NULL == (buf = realloc(buf, sz))) - err(1, "realloc"); - *bufp = buf; - *bufsz = sz; + if (sz > blk->sz) { + blk->buf = realloc(blk->buf, sz); + if (NULL == blk->buf) { + perror(NULL); + exit(EXIT_FAILURE); + } + blk->sz = sz; } - /* - * Fill buf with file blocksize and parse newlines into ln. - */ + /* Fill buf with file blocksize. */ - for (lnn = 1, pos = 0; ; ) { - if (-1 == (ssz = read(fd, buf, sz))) { - warn("%s", f); - return(0); + for (lnn = pos = comment = 0; ; ) { + if (-1 == (ssz = read(curp->fd, blk->buf, sz))) { + perror(curp->file); + return(-1); } else if (0 == ssz) break; + /* Parse the read block into partial or full lines. */ + for (i = 0; i < (int)ssz; i++) { - if (pos >= (int)*lnsz) { - *lnsz += 256; /* Step-size. */ - ln = realloc(ln, *lnsz); - if (NULL == ln) - err(1, "realloc"); - *lnp = ln; + if (pos >= (int)ln->sz) { + ln->sz += 256; /* Step-size. */ + ln->buf = realloc(ln->buf, ln->sz); + if (NULL == ln->buf) { + perror(NULL); + return(EXIT_FAILURE); + } } - if ('\n' != buf[i]) { - ln[pos++] = buf[i]; + if ('\n' != blk->buf[i]) { + if (comment) + continue; + ln->buf[pos++] = blk->buf[i]; + + /* Handle in-line `\"' comments. */ + + if (1 == pos || '\"' != ln->buf[pos - 1]) + continue; + + for (j = pos - 2; j >= 0; j--) + if ('\\' != ln->buf[j]) + break; + + if ( ! ((pos - 2 - j) % 2)) + continue; + + comment = 1; + pos -= 2; continue; - } + } - /* Check for CPP-escaped newline. */ + /* Handle escaped `\\n' newlines. */ - if (pos > 0 && '\\' == ln[pos - 1]) { + if (pos > 0 && 0 == comment && + '\\' == ln->buf[pos - 1]) { for (j = pos - 1; j >= 0; j--) - if ('\\' != ln[j]) + if ('\\' != ln->buf[j]) break; - if ( ! ((pos - j) % 2)) { pos--; lnn++; @@ -296,34 +388,166 @@ fdesc(char **lnp, size_t *lnsz, char **bufp, size_t *b } } - ln[pos] = 0; - if ( ! mdoc_parseln(mdoc, lnn, ln)) - return(0); + ln->buf[pos] = 0; lnn++; - pos = 0; + + /* If unset, assign parser in pset(). */ + + if ( ! (man || mdoc) && ! pset(ln->buf, + pos, curp, &man, &mdoc)) + return(-1); + + pos = comment = 0; + + /* Pass down into parsers. */ + + if (man && ! man_parseln(man, lnn, ln->buf)) + return(0); + if (mdoc && ! mdoc_parseln(mdoc, lnn, ln->buf)) + return(0); } } - return(mdoc_endparse(mdoc)); + /* NOTE a parser may not have been assigned, yet. */ + + if ( ! (man || mdoc)) { + fprintf(stderr, "%s: Not a manual\n", curp->file); + return(0); + } + + if (mdoc && ! mdoc_endparse(mdoc)) + return(0); + if (man && ! man_endparse(man)) + return(0); + + /* If unset, allocate output dev now (if applicable). */ + + if ( ! (curp->outman && curp->outmdoc)) { + switch (curp->outtype) { + case (OUTT_HTML): + curp->outdata = html_alloc(curp->outopts); + curp->outman = html_man; + curp->outmdoc = html_mdoc; + curp->outfree = html_free; + break; + case (OUTT_TREE): + curp->outman = tree_man; + curp->outmdoc = tree_mdoc; + break; + case (OUTT_LINT): + break; + default: + curp->outdata = ascii_alloc(); + curp->outman = terminal_man; + curp->outmdoc = terminal_mdoc; + curp->outfree = terminal_free; + break; + } + } + + /* Execute the out device, if it exists. */ + + if (man && curp->outman) + (*curp->outman)(curp->outdata, man); + if (mdoc && curp->outmdoc) + (*curp->outmdoc)(curp->outdata, mdoc); + + return(1); } static int +pset(const char *buf, int pos, struct curparse *curp, + struct man **man, struct mdoc **mdoc) +{ + int i; + + /* + * Try to intuit which kind of manual parser should be used. If + * passed in by command-line (-man, -mdoc), then use that + * explicitly. If passed as -mandoc, then try to guess from the + * line: either skip dot-lines, use -mdoc when finding `.Dt', or + * default to -man, which is more lenient. + */ + + if (buf[0] == '.') { + for (i = 1; buf[i]; i++) + if (' ' != buf[i] && '\t' != buf[i]) + break; + if (0 == buf[i]) + return(1); + } + + switch (curp->inttype) { + case (INTT_MDOC): + if (NULL == curp->mdoc) + curp->mdoc = mdoc_init(curp); + if (NULL == (*mdoc = curp->mdoc)) + return(0); + curp->lastmdoc = *mdoc; + return(1); + case (INTT_MAN): + if (NULL == curp->man) + curp->man = man_init(curp); + if (NULL == (*man = curp->man)) + return(0); + curp->lastman = *man; + return(1); + default: + break; + } + + if (pos >= 3 && 0 == memcmp(buf, ".Dd", 3)) { + if (NULL == curp->mdoc) + curp->mdoc = mdoc_init(curp); + if (NULL == (*mdoc = curp->mdoc)) + return(0); + curp->lastmdoc = *mdoc; + return(1); + } + + if (NULL == curp->man) + curp->man = man_init(curp); + if (NULL == (*man = curp->man)) + return(0); + curp->lastman = *man; + return(1); +} + + +static int +moptions(enum intt *tflags, char *arg) +{ + + if (0 == strcmp(arg, "doc")) + *tflags = INTT_MDOC; + else if (0 == strcmp(arg, "andoc")) + *tflags = INTT_AUTO; + else if (0 == strcmp(arg, "an")) + *tflags = INTT_MAN; + else { + fprintf(stderr, "%s: Bad argument", arg); + return(0); + } + + return(1); +} + + +static int toptions(enum outt *tflags, char *arg) { if (0 == strcmp(arg, "ascii")) *tflags = OUTT_ASCII; - else if (0 == strcmp(arg, "latin1")) - *tflags = OUTT_LATIN1; - else if (0 == strcmp(arg, "utf8")) - *tflags = OUTT_UTF8; else if (0 == strcmp(arg, "lint")) *tflags = OUTT_LINT; else if (0 == strcmp(arg, "tree")) *tflags = OUTT_TREE; + else if (0 == strcmp(arg, "html")) + *tflags = OUTT_HTML; else { - warnx("bad argument: -T%s", arg); + fprintf(stderr, "%s: Bad argument", arg); return(0); } @@ -331,75 +555,80 @@ toptions(enum outt *tflags, char *arg) } -/* - * Parse out the options for [-fopt...] setting compiler options. These - * can be comma-delimited or called again. - */ static int foptions(int *fflags, char *arg) { - char *v; - char *toks[4]; + char *v, *o; + const char *toks[8]; toks[0] = "ign-scope"; - toks[1] = "ign-escape"; - toks[2] = "ign-macro"; - toks[3] = NULL; + toks[1] = "no-ign-escape"; + toks[2] = "no-ign-macro"; + toks[3] = "no-ign-chars"; + toks[4] = "ign-errors"; + toks[5] = "strict"; + toks[6] = "ign-escape"; + toks[7] = NULL; - while (*arg) - switch (getsubopt(&arg, toks, &v)) { + while (*arg) { + o = arg; + switch (getsubopt(&arg, UNCONST(toks), &v)) { case (0): - *fflags |= MDOC_IGN_SCOPE; + *fflags |= IGN_SCOPE; break; case (1): - *fflags |= MDOC_IGN_ESCAPE; + *fflags |= NO_IGN_ESCAPE; break; case (2): - *fflags |= MDOC_IGN_MACRO; + *fflags |= NO_IGN_MACRO; break; + case (3): + *fflags |= NO_IGN_CHARS; + break; + case (4): + *fflags |= IGN_ERRORS; + break; + case (5): + *fflags |= NO_IGN_ESCAPE | + NO_IGN_MACRO | NO_IGN_CHARS; + break; + case (6): + *fflags &= ~NO_IGN_ESCAPE; + break; default: - warnx("bad argument: -f%s", arg); + fprintf(stderr, "%s: Bad argument", o); return(0); } + } return(1); } -/* - * Parse out the options for [-Werr...], which sets warning modes. - * These can be comma-delimited or called again. - */ static int woptions(int *wflags, char *arg) { - char *v; - char *toks[5]; + char *v, *o; + const char *toks[3]; toks[0] = "all"; - toks[1] = "compat"; - toks[2] = "syntax"; - toks[3] = "error"; - toks[4] = NULL; + toks[1] = "error"; + toks[2] = NULL; - while (*arg) - switch (getsubopt(&arg, toks, &v)) { + while (*arg) { + o = arg; + switch (getsubopt(&arg, UNCONST(toks), &v)) { case (0): *wflags |= WARN_WALL; break; case (1): - *wflags |= WARN_WCOMPAT; - break; - case (2): - *wflags |= WARN_WSYNTAX; - break; - case (3): *wflags |= WARN_WERR; break; default: - warnx("bad argument: -W%s", arg); + fprintf(stderr, "%s: Bad argument", o); return(0); } + } return(1); } @@ -409,45 +638,33 @@ woptions(int *wflags, char *arg) static int merr(void *arg, int line, int col, const char *msg) { + struct curparse *curp; - warnx("error: %s (line %d, column %d)", msg, line, col); + curp = (struct curparse *)arg; + + (void)fprintf(stderr, "%s:%d:%d: error: %s\n", + curp->file, line, col + 1, msg); + return(0); } static int -mwarn(void *arg, int line, int col, - enum mdoc_warn type, const char *msg) +mwarn(void *arg, int line, int col, const char *msg) { - int flags; - char *wtype; + struct curparse *curp; - flags = *(int *)arg; - wtype = NULL; + curp = (struct curparse *)arg; - switch (type) { - case (WARN_COMPAT): - wtype = "compat"; - if (flags & WARN_WCOMPAT) - break; + if ( ! (curp->wflags & WARN_WALL)) return(1); - case (WARN_SYNTAX): - wtype = "syntax"; - if (flags & WARN_WSYNTAX) - break; - return(1); - } - assert(wtype); - warnx("%s warning: %s (line %d, column %d)", - wtype, msg, line, col); + (void)fprintf(stderr, "%s:%d:%d: warning: %s\n", + curp->file, line, col + 1, msg); - if ( ! (flags & WARN_WERR)) + if ( ! (curp->wflags & WARN_WERR)) return(1); - - warnx("%s: considering warnings as errors", - __progname); + return(0); } -