]> git.cameronkatri.com Git - mandoc.git/blobdiff - main.c
Lint-check (removed unused variable).
[mandoc.git] / main.c
diff --git a/main.c b/main.c
index 286b5bd630a7910e435eaeaa65d7df5968a3bfef..26988773e7bf647641613f7338ba8750fd7e60e9 100644 (file)
--- a/main.c
+++ b/main.c
@@ -1,4 +1,4 @@
-/*     $Id: main.c,v 1.29 2009/06/16 19:13:28 kristaps Exp $ */
+/*     $Id: main.c,v 1.38 2009/07/07 09:52:08 kristaps Exp $ */
 /*
  * Copyright (c) 2008, 2009 Kristaps Dzonsons <kristaps@kth.se>
  *
@@ -65,9 +65,7 @@ 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_WALL        (1 << 0)       /* All-warnings mask. */
 #define        WARN_WERR        (1 << 2)       /* Warnings->errors. */
        int               fflags;
 #define        IGN_SCOPE        (1 << 0)       /* Ignore scope errors. */
@@ -98,11 +96,7 @@ 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               manwarn(void *, int, int, const char *);
-static int               mdocwarn(void *, int, int, 
-                               enum mdoc_warn, const char *);
-static int               fstdin(struct buf *, struct buf *, 
-                               struct curparse *);
+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 *,
@@ -134,19 +128,19 @@ main(int argc, char *argv[])
                switch (c) {
                case ('f'):
                        if ( ! foptions(&curp.fflags, optarg))
-                               return(0);
+                               return(EXIT_FAILURE);
                        break;
                case ('m'):
                        if ( ! moptions(&curp.inttype, optarg))
-                               return(0);
+                               return(EXIT_FAILURE);
                        break;
                case ('T'):
                        if ( ! toptions(&curp.outtype, optarg))
-                               return(0);
+                               return(EXIT_FAILURE);
                        break;
                case ('W'):
                        if ( ! woptions(&curp.wflags, optarg))
-                               return(0);
+                               return(EXIT_FAILURE);
                        break;
                case ('V'):
                        version();
@@ -159,16 +153,17 @@ main(int argc, char *argv[])
        argc -= optind;
        argv += optind;
 
-       /* Configure buffers. */
-
        bzero(&ln, sizeof(struct buf));
        bzero(&blk, sizeof(struct buf));
 
        rc = 1;
 
-       if (NULL == *argv)
-               if ( ! fstdin(&blk, &ln, &curp))
+       if (NULL == *argv) {
+               curp.file = "<stdin>";
+               curp.fd = STDIN_FILENO;
+               if ( ! fdesc(&blk, &ln, &curp))
                        rc = 0;
+       }
 
        while (rc && *argv) {
                if ( ! ffile(&blk, &ln, *argv, &curp))
@@ -190,8 +185,6 @@ main(int argc, char *argv[])
                free(blk.buf);
        if (ln.buf)
                free(ln.buf);
-
-       /* TODO: have a curp_free routine. */
        if (curp.outfree)
                (*curp.outfree)(curp.outdata);
        if (curp.mdoc)
@@ -231,19 +224,18 @@ man_init(struct curparse *curp)
        struct man_cb    mancb;
 
        mancb.man_err = merr;
-       mancb.man_warn = manwarn;
+       mancb.man_warn = mwarn;
 
-       /*
-        * Default behaviour is to ignore unknown macros.  This is
-        * specified in mandoc.1.
-        */
+       /* Defaults from mandoc.1. */
 
-       pflags = MAN_IGN_MACRO;
-
-       /* Override default behaviour... */
+       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");
@@ -260,18 +252,12 @@ mdoc_init(struct curparse *curp)
        struct mdoc_cb   mdoccb;
 
        mdoccb.mdoc_err = merr;
-       mdoccb.mdoc_warn = mdocwarn;
+       mdoccb.mdoc_warn = mwarn;
 
-       /* 
-        * Default behaviour is to ignore unknown macros, escape
-        * sequences and characters (very liberal).  This is specified
-        * in mandoc.1.
-        */
+       /* Defaults from 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)
@@ -288,16 +274,6 @@ mdoc_init(struct curparse *curp)
 }
 
 
-static int
-fstdin(struct buf *blk, struct buf *ln, struct curparse *curp)
-{
-
-       curp->file = "<stdin>";
-       curp->fd = STDIN_FILENO;
-       return(fdesc(blk, ln, curp));
-}
-
-
 static int
 ffile(struct buf *blk, struct buf *ln, 
                const char *file, struct curparse *curp)
@@ -340,7 +316,7 @@ fdesc(struct buf *blk, struct buf *ln, struct curparse *curp)
         */
 
        if (-1 == fstat(curp->fd, &st))
-               warnx("%s", curp->file);
+               warn("%s", curp->file);
        else if ((size_t)st.st_blksize > sz)
                sz = st.st_blksize;
 
@@ -421,6 +397,8 @@ fdesc(struct buf *blk, struct buf *ln, struct curparse *curp)
 
                        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))
@@ -482,7 +460,7 @@ pset(const char *buf, int pos, struct curparse *curp,
         * 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
+        * line: either skip dot-lines, use -mdoc when finding `.Dt', or
         * default to -man, which is more lenient.
         */
 
@@ -569,14 +547,10 @@ 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            *v, *o;
        char            *toks[6];
 
        toks[0] = "ign-scope";
@@ -586,7 +560,8 @@ foptions(int *fflags, char *arg)
        toks[4] = "strict";
        toks[5] = NULL;
 
-       while (*arg) 
+       while (*arg) {
+               o = arg;
                switch (getsubopt(&arg, toks, &v)) {
                case (0):
                        *fflags |= IGN_SCOPE;
@@ -605,48 +580,39 @@ foptions(int *fflags, char *arg)
                                   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);
 }
@@ -659,77 +625,31 @@ merr(void *arg, int line, int col, const char *msg)
        struct curparse *curp;
 
        curp = (struct curparse *)arg;
+
        warnx("%s:%d: error: %s (column %d)", 
                        curp->file, line, msg, col);
 
-       /* Always exit on errors... */
        return(0);
 }
 
 
 static int
-mdocwarn(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;
-               return(1);
-       case (WARN_SYNTAX):
-               wtype = "syntax";
-               if (curp->wflags & WARN_WSYNTAX)
-                       break;
+       if ( ! (curp->wflags & WARN_WALL))
                return(1);
-       }
 
-       assert(wtype);
-       warnx("%s:%d: %s warning: %s (column %d)", 
-                       curp->file, line, wtype, msg, col);
+       warnx("%s:%d: 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);
+       warnx("considering warnings as errors");
        return(0);
 }
 
-
-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);
-}