-/* $Id: main.c,v 1.4 2009/03/19 18:30:26 kristaps Exp $ */
+/* $Id: main.c,v 1.28 2009/06/15 10:36:01 kristaps Exp $ */
/*
- * Copyright (c) 2008, 2009 Kristaps Dzonsons <kristaps@openbsd.org>
+ * Copyright (c) 2008, 2009 Kristaps Dzonsons <kristaps@kth.se>
*
* 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 <sys/stat.h>
#include <unistd.h>
#include "mdoc.h"
+#include "man.h"
+
+/* Account for FreeBSD and Linux in our declarations. */
#ifdef __linux__
extern int getsubopt(char **, char * const *, char **);
# ifndef __dead
# define __dead __attribute__((__noreturn__))
# endif
+#elif defined(__dead2)
+# ifndef __dead
+# define __dead __dead2
+# endif
#endif
-#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. */
+typedef int (*out_mdoc)(void *, const struct mdoc *);
+typedef int (*out_man)(void *, const struct man *);
+typedef void (*out_free)(void *);
+
+struct buf {
+ char *buf;
+ size_t sz;
+};
+
+enum intt {
+ INTT_AUTO,
+ INTT_MDOC,
+ INTT_MAN
+};
-enum outt {
- OUTT_ASCII,
- OUTT_LATIN1,
- OUTT_UTF8,
+enum outt {
+ OUTT_ASCII = 0,
OUTT_TREE,
OUTT_LINT
};
-typedef int (*out_run)(void *, const struct mdoc *);
-typedef void (*out_free)(void *);
-
-extern char *__progname;
+struct curparse {
+ const char *file; /* Current parse. */
+ int fd; /* Current parse. */
+ int wflags;
+#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. */
+ 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. */
+ 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;
+};
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 int tree_mdoc(void *, const struct mdoc *);
+extern int tree_man(void *, const struct man *);
+extern int terminal_mdoc(void *, const struct mdoc *);
+extern int terminal_man(void *, const struct man *);
extern void terminal_free(void *);
-__dead static void version(void);
-__dead static void usage(void);
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,
+static int manwarn(void *, int, int, const char *);
+static int mdocwarn(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 fstdin(struct buf *, struct buf *,
+ struct curparse *);
+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;
+
+ bzero(&curp, sizeof(struct curparse));
- fflags = wflags = 0;
- outtype = OUTT_ASCII;
+ 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:VW:T:")))
switch (c) {
case ('f'):
- if ( ! foptions(&fflags, optarg))
+ if ( ! foptions(&curp.fflags, optarg))
+ return(0);
+ break;
+ case ('m'):
+ if ( ! moptions(&curp.inttype, optarg))
return(0);
break;
case ('T'):
- if ( ! toptions(&outtype, optarg))
+ if ( ! toptions(&curp.outtype, optarg))
return(0);
break;
case ('W'):
- if ( ! woptions(&wflags, optarg))
+ if ( ! woptions(&curp.wflags, optarg))
return(0);
break;
case ('V'):
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.
- */
-
- 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;
+ /* Configure buffers. */
+
+ bzero(&ln, sizeof(struct buf));
+ bzero(&blk, sizeof(struct buf));
+
+ rc = 1;
+
+ if (NULL == *argv)
+ if ( ! fstdin(&blk, &ln, &curp))
+ rc = 0;
+
+ while (rc && *argv) {
+ if ( ! ffile(&blk, &ln, *argv, &curp))
+ rc = 0;
+ argv++;
+ if (*argv && rc) {
+ if (curp.lastman)
+ if ( ! man_reset(curp.lastman))
+ rc = 0;
+ if (curp.lastmdoc)
+ if ( ! mdoc_reset(curp.lastmdoc))
+ rc = 0;
+ curp.lastman = NULL;
+ curp.lastmdoc = NULL;
+ }
}
- /*
- * All callbacks route into here, where we print them onto the
- * screen. XXX - for now, no path for debugging messages.
- */
+ if (blk.buf)
+ free(blk.buf);
+ if (ln.buf)
+ free(ln.buf);
- cb.mdoc_msg = NULL;
- cb.mdoc_err = merr;
- cb.mdoc_warn = mwarn;
+ /* TODO: have a curp_free routine. */
+ if (curp.outfree)
+ (*curp.outfree)(curp.outdata);
+ if (curp.mdoc)
+ mdoc_free(curp.mdoc);
+ if (curp.man)
+ man_free(curp.man);
- buf = line = NULL;
- bufsz = linesz = 0;
+ return(rc ? EXIT_SUCCESS : EXIT_FAILURE);
+}
- mdoc = mdoc_alloc(&wflags, fflags, &cb);
+
+__dead static void
+version(void)
+{
+
+ (void)printf("%s %s\n", __progname, VERSION);
+ exit(EXIT_SUCCESS);
+}
+
+
+__dead static void
+usage(void)
+{
+
+ (void)fprintf(stderr, "usage: %s [-V] [-foption...] "
+ "[-mformat] [-Toutput] [-Werr...]\n",
+ __progname);
+ exit(EXIT_FAILURE);
+}
+
+
+static struct man *
+man_init(struct curparse *curp)
+{
+ int pflags;
+ struct man *man;
+ struct man_cb mancb;
+
+ mancb.man_err = merr;
+ mancb.man_warn = manwarn;
/*
- * Loop around available files.
+ * Default behaviour is to ignore unknown macros. This is
+ * specified in mandoc.1.
*/
- if (NULL == *argv) {
- c = fdesc(&line, &linesz, &buf, &bufsz,
- "stdin", STDIN_FILENO, mdoc);
- rc = 0;
- if (c && NULL == outrun)
- rc = 1;
- else if (c && outrun && (*outrun)(outdata, mdoc))
- rc = 1;
- } else {
- while (*argv) {
- c = file(&line, &linesz, &buf,
- &bufsz, *argv, mdoc);
- if ( ! c)
- break;
- if (outrun && ! (*outrun)(outdata, mdoc))
- break;
- /* Reset the parser for another file. */
- mdoc_reset(mdoc);
- argv++;
- }
- rc = NULL == *argv;
- }
+ pflags = MAN_IGN_MACRO;
- if (buf)
- free(buf);
- if (line)
- free(line);
- if (outfree)
- (*outfree)(outdata);
+ /* Override default behaviour... */
- mdoc_free(mdoc);
+ if (curp->fflags & NO_IGN_MACRO)
+ pflags &= ~MAN_IGN_MACRO;
- return(rc ? EXIT_SUCCESS : EXIT_FAILURE);
+ if (NULL == (man = man_alloc(curp, pflags, &mancb)))
+ warnx("memory exhausted");
+
+ return(man);
}
-__dead static void
-version(void)
+static struct mdoc *
+mdoc_init(struct curparse *curp)
{
+ int pflags;
+ struct mdoc *mdoc;
+ struct mdoc_cb mdoccb;
- (void)printf("%s %s\n", __progname, VERSION);
- exit(0);
- /* NOTREACHED */
+ mdoccb.mdoc_err = merr;
+ mdoccb.mdoc_warn = mdocwarn;
+
+ /*
+ * Default behaviour is to ignore unknown macros, escape
+ * sequences and characters (very liberal). This is specified
+ * in mandoc.1.
+ */
+
+ pflags = MDOC_IGN_MACRO | MDOC_IGN_ESCAPE | MDOC_IGN_CHARS;
+
+ /* Override default behaviour... */
+
+ 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;
+
+ if (NULL == (mdoc = mdoc_alloc(curp, pflags, &mdoccb)))
+ warnx("memory exhausted");
+
+ return(mdoc);
}
-__dead static void
-usage(void)
+static int
+fstdin(struct buf *blk, struct buf *ln, struct curparse *curp)
{
- (void)fprintf(stderr, "usage: %s\n", __progname);
- exit(1);
- /* NOTREACHED */
+ curp->file = "<stdin>";
+ curp->fd = STDIN_FILENO;
+ return(fdesc(blk, ln, curp));
}
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);
+ curp->file = file;
+ if (-1 == (curp->fd = open(curp->file, O_RDONLY, 0))) {
+ warn("%s", curp->file);
return(0);
}
- 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))
+ warn("%s", 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;
+ 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 (sz > *bufsz) {
- if (NULL == (buf = realloc(buf, sz)))
- err(1, "realloc");
- *bufp = buf;
- *bufsz = sz;
+ if (-1 == fstat(curp->fd, &st))
+ warnx("%s", curp->file);
+ else if ((size_t)st.st_blksize > sz)
+ sz = st.st_blksize;
+
+ if (sz > blk->sz) {
+ blk->buf = realloc(blk->buf, sz);
+ if (NULL == blk->buf) {
+ warn("realloc");
+ return(0);
+ }
+ 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);
+ for (lnn = 0, pos = 0; ; ) {
+ if (-1 == (ssz = read(curp->fd, blk->buf, sz))) {
+ warn("%s", curp->file);
return(0);
} 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) {
+ warn("realloc");
+ return(0);
+ }
}
- if ('\n' != buf[i]) {
- ln[pos++] = buf[i];
+ if ('\n' != blk->buf[i]) {
+ ln->buf[pos++] = blk->buf[i];
continue;
}
- /* Check for CPP-escaped newline. */
+ /* Check for CPP-escaped newline. */
- if (pos > 0 && '\\' == ln[pos - 1]) {
+ if (pos > 0 && '\\' == ln->buf[pos - 1]) {
for (j = pos - 1; j >= 0; j--)
- if ('\\' != ln[j])
+ if ('\\' != ln->buf[j])
break;
if ( ! ((pos - j) % 2)) {
}
}
- ln[pos] = 0;
- if ( ! mdoc_parseln(mdoc, lnn, ln))
- return(0);
+ ln->buf[pos] = 0;
lnn++;
+
+ /*
+ * If no manual parser has been assigned, then
+ * try to assign one in pset(), which may do
+ * nothing at all. After this, parse the manual
+ * line accordingly.
+ */
+
+ if ( ! (man || mdoc) && ! pset(ln->buf,
+ pos, curp, &man, &mdoc))
+ return(0);
+
pos = 0;
+
+ 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 that a parser may not have been assigned, yet. */
+
+ if ( ! (man || mdoc)) {
+ warnx("%s: not a manual", curp->file);
+ return(0);
+ }
+
+ if (mdoc && ! mdoc_endparse(mdoc))
+ return(0);
+ if (man && ! man_endparse(man))
+ return(0);
+
+ /*
+ * If an output device hasn't been allocated, see if we should
+ * do so now. Note that not all outtypes have functions, so
+ * this switch statement may be superfluous, but it's
+ * low-overhead enough not to matter very much.
+ */
+
+ if ( ! (curp->outman && curp->outmdoc)) {
+ switch (curp->outtype) {
+ 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)
+ if ( ! (*curp->outman)(curp->outdata, man))
+ return(0);
+ if (mdoc && curp->outmdoc)
+ if ( ! (*curp->outmdoc)(curp->outdata, mdoc))
+ return(0);
+
+ return(1);
+}
+
+
+static int
+pset(const char *buf, int pos, struct curparse *curp,
+ struct man **man, struct mdoc **mdoc)
+{
+
+ /*
+ * 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 comments, use -mdoc when finding `.Dt', or
+ * default to -man, which is more lenient.
+ */
+
+ if (pos >= 3 && 0 == memcmp(buf, ".\\\"", 3))
+ 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 {
+ warnx("bad argument: -m%s", arg);
+ return(0);
+ }
+
+ return(1);
}
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"))
foptions(int *fflags, char *arg)
{
char *v;
- char *toks[4];
+ char *toks[6];
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] = "strict";
+ toks[5] = NULL;
while (*arg)
switch (getsubopt(&arg, 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 |= NO_IGN_ESCAPE |
+ NO_IGN_MACRO | NO_IGN_CHARS;
break;
default:
warnx("bad argument: -f%s", 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;
+ warnx("%s:%d: error: %s (column %d)",
+ curp->file, line, msg, col);
+
+ /* Always exit on errors... */
return(0);
}
static int
-mwarn(void *arg, int line, int col,
+mdocwarn(void *arg, int line, int col,
enum mdoc_warn type, const char *msg)
{
- int flags;
+ struct curparse *curp;
char *wtype;
- flags = *(int *)arg;
+ curp = (struct curparse *)arg;
wtype = NULL;
switch (type) {
case (WARN_COMPAT):
wtype = "compat";
- if (flags & WARN_WCOMPAT)
+ if (curp->wflags & WARN_WCOMPAT)
break;
return(1);
case (WARN_SYNTAX):
wtype = "syntax";
- if (flags & WARN_WSYNTAX)
+ if (curp->wflags & WARN_WSYNTAX)
break;
return(1);
}
assert(wtype);
- warnx("%s warning: %s (line %d, column %d)",
- wtype, msg, line, col);
+ warnx("%s:%d: %s warning: %s (column %d)",
+ curp->file, line, wtype, msg, col);
- if ( ! (flags & WARN_WERR))
+ if ( ! (curp->wflags & WARN_WERR))
return(1);
+
+ /*
+ * If the -Werror flag is passed in, as in gcc, then all
+ * warnings are considered as errors.
+ */
warnx("%s: considering warnings as errors",
__progname);
}
+static int
+manwarn(void *arg, int line, int col, const char *msg)
+{
+ struct curparse *curp;
+
+ curp = (struct curparse *)arg;
+
+ if ( ! (curp->wflags & WARN_WSYNTAX))
+ return(1);
+
+ warnx("%s:%d: syntax warning: %s (column %d)",
+ curp->file, line, msg, col);
+
+ if ( ! (curp->wflags & WARN_WERR))
+ return(1);
+
+ /*
+ * If the -Werror flag is passed in, as in gcc, then all
+ * warnings are considered as errors.
+ */
+
+ warnx("%s: considering warnings as errors",
+ __progname);
+ return(0);
+}
git.ckatri.com / mandoc.git / blobdiff