-/* $Id: main.c,v 1.10 2009/03/23 14:22:11 kristaps Exp $ */
+/* $Id: main.c,v 1.44 2009/09/21 13:06:13 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 "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(__FreeBSD__)
+#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 *);
+
struct buf {
char *buf;
size_t sz;
};
-struct curparse {
- const char *file;
- 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. */
-};
-
enum intt {
- INTT_MDOC = 0,
+ INTT_AUTO,
+ INTT_MDOC,
INTT_MAN
};
enum outt {
OUTT_ASCII = 0,
- OUTT_LATIN1,
- OUTT_UTF8,
OUTT_TREE,
+ OUTT_HTML,
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 (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;
+};
+extern void *html_alloc(char *);
+extern void html_mdoc(void *, const struct mdoc *);
+extern void html_man(void *, const struct man *);
+extern void html_free(void *);
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 tree_mdoc(void *, const struct mdoc *);
+extern void tree_man(void *, const struct man *);
+extern void terminal_mdoc(void *, const struct mdoc *);
+extern void terminal_man(void *, const struct man *);
extern void terminal_free(void *);
static int foptions(int *, 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(struct buf *, struct buf *,
- const char *,
- struct man *, 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 *,
- const char *, int,
- struct man *, struct mdoc *);
-
+ 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;
- struct mdoc_cb cb;
- struct man *man;
- struct mdoc *mdoc;
- void *outdata;
- enum outt outtype;
- enum intt inttype;
+ int c, rc;
struct buf ln, blk;
- out_run outrun;
- out_free outfree;
struct curparse curp;
- fflags = 0;
- outtype = OUTT_ASCII;
- inttype = INTT_MDOC;
-
bzero(&curp, sizeof(struct curparse));
+ curp.inttype = INTT_AUTO;
+ curp.outtype = OUTT_ASCII;
+
/* LINTED */
- while (-1 != (c = getopt(argc, argv, "f:m: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(&inttype, optarg))
- return(0);
+ if ( ! moptions(&curp.inttype, optarg))
+ return(EXIT_FAILURE);
+ break;
+ case ('o'):
+ curp.outopts = optarg;
break;
case ('T'):
- if ( ! toptions(&outtype, optarg))
- return(0);
+ if ( ! toptions(&curp.outtype, optarg))
+ return(EXIT_FAILURE);
break;
case ('W'):
if ( ! woptions(&curp.wflags, optarg))
- return(0);
+ return(EXIT_FAILURE);
break;
case ('V'):
version();
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;
- }
-
- /*
- * All callbacks route into here, where we print them onto the
- * screen. XXX - for now, no path for debugging messages.
- */
-
- cb.mdoc_msg = NULL;
- cb.mdoc_err = merr;
- cb.mdoc_warn = mwarn;
-
bzero(&ln, sizeof(struct buf));
bzero(&blk, sizeof(struct buf));
- man = NULL;
- mdoc = NULL;
-
- switch (inttype) {
- case (INTT_MAN):
- man = man_alloc();
- break;
- default:
- mdoc = mdoc_alloc(&curp, fflags, &cb);
- break;
- }
-
- /*
- * Loop around available files.
- */
+ rc = 1;
if (NULL == *argv) {
curp.file = "<stdin>";
- rc = 0;
- c = fdesc(&blk, &ln, "stdin",
- STDIN_FILENO, man, mdoc);
-
- if (c && NULL == outrun)
- rc = 1;
-#if 0
- else if (c && outrun && (*outrun)(outdata, mdoc))
- rc = 1;
-#endif
- } else {
- while (*argv) {
- curp.file = *argv;
- c = file(&blk, &ln, *argv, man, mdoc);
- if ( ! c)
- break;
-#if 0
- if (outrun && ! (*outrun)(outdata, mdoc))
- break;
-#endif
- if (man)
- man_reset(man);
- if (mdoc)
- mdoc_reset(mdoc);
+ curp.fd = STDIN_FILENO;
+
+ c = fdesc(&blk, &ln, &curp);
+ if ( ! (IGN_ERRORS & curp.fflags))
+ rc = 1 == c ? 1 : 0;
+ else
+ rc = -1 == c ? 0 : 1;
+ }
- argv++;
+ 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;
+
+ 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;
}
- rc = NULL == *argv;
}
if (blk.buf)
free(blk.buf);
if (ln.buf)
free(ln.buf);
- if (outfree)
- (*outfree)(outdata);
- if (mdoc)
- mdoc_free(mdoc);
- if (man)
- man_free(man);
+ if (curp.outfree)
+ (*curp.outfree)(curp.outdata);
+ if (curp.mdoc)
+ mdoc_free(curp.mdoc);
+ if (curp.man)
+ man_free(curp.man);
return(rc ? EXIT_SUCCESS : EXIT_FAILURE);
}
{
(void)printf("%s %s\n", __progname, VERSION);
- exit(0);
- /* NOTREACHED */
+ exit(EXIT_SUCCESS);
}
usage(void)
{
- (void)fprintf(stderr, "usage: %s\n", __progname);
- exit(1);
- /* NOTREACHED */
+ (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 = 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;
+
+ if (NULL == (man = man_alloc(curp, pflags, &mancb)))
+ warnx("memory exhausted");
+
+ return(man);
+}
+
+
+static struct mdoc *
+mdoc_init(struct curparse *curp)
+{
+ int pflags;
+ struct mdoc *mdoc;
+ 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;
+
+ if (NULL == (mdoc = mdoc_alloc(curp, pflags, &mdoccb)))
+ warnx("memory exhausted");
+
+ return(mdoc);
}
static int
-file(struct buf *blk, struct buf *ln, const char *file,
- struct man *man, 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))) {
+ warn("%s", curp->file);
+ return(-1);
}
- c = fdesc(blk, ln, file, fd, man, 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(struct buf *blk, struct buf *ln,
- const char *f, int fd,
- struct man *man, 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;
+ int j, i, pos, lnn, comment;
+ struct man *man;
+ struct mdoc *mdoc;
- assert( ! (man && mdoc));
+ 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.
*/
- sz = BUFSIZ;
-
- if (-1 == fstat(fd, &st))
- warnx("%s", f);
+ if (-1 == fstat(curp->fd, &st))
+ warn("%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)
- err(1, "realloc");
+ if (NULL == blk->buf) {
+ warn("realloc");
+ return(-1);
+ }
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, blk->buf, sz))) {
- warn("%s", f);
- return(0);
+ for (lnn = pos = comment = 0; ; ) {
+ if (-1 == (ssz = read(curp->fd, blk->buf, sz))) {
+ warn("%s", 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)ln->sz) {
ln->sz += 256; /* Step-size. */
ln->buf = realloc(ln->buf, ln->sz);
- if (NULL == ln->buf)
- err(1, "realloc");
+ if (NULL == ln->buf) {
+ warn("realloc");
+ return(-1);
+ }
}
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->buf[pos - 1]) {
+ if (pos > 0 && 0 == comment &&
+ '\\' == ln->buf[pos - 1]) {
for (j = pos - 1; j >= 0; j--)
if ('\\' != ln->buf[j])
break;
-
if ( ! ((pos - j) % 2)) {
pos--;
lnn++;
}
ln->buf[pos] = 0;
- if (mdoc && ! mdoc_parseln(mdoc, lnn, ln->buf))
- return(0);
+ lnn++;
+
+ /* 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);
- lnn++;
- pos = 0;
+ if (mdoc && ! mdoc_parseln(mdoc, lnn, ln->buf))
+ return(0);
}
}
- if (mdoc)
- return(mdoc_endparse(mdoc));
+ /* NOTE a parser may not have been assigned, yet. */
+
+ if ( ! (man || mdoc)) {
+ (void)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;
+ }
- return(man_endparse(man));
+ 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);
}
moptions(enum intt *tflags, char *arg)
{
- if (0 == strcmp(arg, "mdoc"))
+ if (0 == strcmp(arg, "doc"))
*tflags = INTT_MDOC;
- else if (0 == strcmp(arg, "man"))
+ 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);
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);
return(0);
}
-/*
- * 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;
+ char *toks[7];
toks[0] = "ign-scope";
- toks[1] = "ign-escape";
- toks[2] = "ign-macro";
- toks[3] = NULL;
-
- while (*arg)
+ toks[1] = "no-ign-escape";
+ toks[2] = "no-ign-macro";
+ toks[3] = "no-ign-chars";
+ toks[4] = "ign-errors";
+ toks[5] = "strict";
+ toks[6] = NULL;
+
+ while (*arg) {
+ o = 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 |= IGN_ERRORS;
+ break;
+ case (5):
+ *fflags |= NO_IGN_ESCAPE |
+ NO_IGN_MACRO | NO_IGN_CHARS;
break;
default:
- warnx("bad argument: -f%s", arg);
+ warnx("bad argument: -f%s", 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;
+ 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)
+ while (*arg) {
+ o = arg;
switch (getsubopt(&arg, 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);
+ warnx("bad argument: -W%s", o);
return(0);
}
+ }
return(1);
}
curp = (struct curparse *)arg;
- warnx("%s:%d: error: %s (column %d)",
- curp->file, line, msg, col);
+ (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)
{
struct curparse *curp;
- char *wtype;
curp = (struct curparse *)arg;
- wtype = NULL;
- switch (type) {
- case (WARN_COMPAT):
- wtype = "compat";
- if (curp->wflags & WARN_WCOMPAT)
- break;
+ if ( ! (curp->wflags & WARN_WALL))
return(1);
- case (WARN_SYNTAX):
- wtype = "syntax";
- if (curp->wflags & WARN_WSYNTAX)
- break;
- return(1);
- }
- assert(wtype);
- warnx("%s:%d: %s warning: %s (column %d)",
- curp->file, line, wtype, msg, col);
+ (void)fprintf(stderr, "%s:%d:%d: warning: %s\n",
+ curp->file, line, col + 1, msg);
if ( ! (curp->wflags & WARN_WERR))
return(1);
-
- warnx("%s: considering warnings as errors",
- __progname);
+
return(0);
}
-