]> git.cameronkatri.com Git - mandoc.git/blobdiff - main.c
Do not mistreat empty arguments to font alternating macros
[mandoc.git] / main.c
diff --git a/main.c b/main.c
index c5569f6172d044cd2d3d544edd0d482d9087d7e2..61e90fe581f52d22c0e95a20a2e31c2c7683b34f 100644 (file)
--- a/main.c
+++ b/main.c
@@ -1,4 +1,4 @@
-/*     $Id: main.c,v 1.223 2015/03/06 09:24:59 kristaps Exp $ */
+/*     $Id: main.c,v 1.232 2015/04/03 08:46:17 schwarze Exp $ */
 /*
  * Copyright (c) 2008-2012 Kristaps Dzonsons <kristaps@bsd.lv>
  * Copyright (c) 2010-2012, 2014, 2015 Ingo Schwarze <schwarze@openbsd.org>
@@ -8,9 +8,9 @@
  * 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
+ * THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHORS DISCLAIM ALL WARRANTIES
  * WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
- * MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
+ * MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHORS 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
 
 #include <sys/types.h>
 #include <sys/param.h> /* MACHINE */
+#include <sys/wait.h>
 
 #include <assert.h>
 #include <ctype.h>
 #include <errno.h>
 #include <fcntl.h>
 #include <glob.h>
+#include <signal.h>
 #include <stdio.h>
 #include <stdint.h>
 #include <stdlib.h>
 #include <string.h>
 #include <unistd.h>
 
-#include "mandoc.h"
 #include "mandoc_aux.h"
-#include "main.h"
+#include "mandoc.h"
+#include "roff.h"
 #include "mdoc.h"
 #include "man.h"
-#include "manpath.h"
+#include "main.h"
+#include "manconf.h"
 #include "mansearch.h"
 
 #if !defined(__GNUC__) || (__GNUC__ < 2)
@@ -81,7 +84,7 @@ struct        curparse {
        out_man           outman;       /* man output ptr */
        out_free          outfree;      /* free output ptr */
        void             *outdata;      /* data for output */
-       char              outopts[BUFSIZ]; /* buf of output opts */
+       struct manoutput *outopts;      /* output options */
 };
 
 static int               fs_lookup(const struct manpaths *,
@@ -91,6 +94,7 @@ static        int               fs_lookup(const struct manpaths *,
 static void              fs_search(const struct mansearch *,
                                const struct manpaths *, int, char**,
                                struct manpage **, size_t *);
+static void              handle_sigpipe(int);
 static int               koptions(int *, char *);
 #if HAVE_SQLITE3
 int                      mandocdb(int, char**);
@@ -98,10 +102,9 @@ int                   mandocdb(int, char**);
 static int               moptions(int *, char *);
 static void              mmsg(enum mandocerr, enum mandoclevel,
                                const char *, int, int, const char *);
-static void              parse(struct curparse *, int,
-                               const char *, enum mandoclevel *);
-static enum mandoclevel  passthrough(const char *, int, int);
-static void              spawn_pager(void);
+static void              parse(struct curparse *, int, const char *);
+static void              passthrough(const char *, int, int);
+static pid_t             spawn_pager(void);
 static int               toptions(struct curparse *, char *);
 static void              usage(enum argmode) __attribute__((noreturn));
 static int               woptions(struct curparse *, char *);
@@ -110,29 +113,30 @@ static    const int sec_prios[] = {1, 4, 5, 8, 6, 3, 7, 2, 9};
 static char              help_arg[] = "help";
 static char             *help_argv[] = {help_arg, NULL};
 static const char       *progname;
+static enum mandoclevel  rc;
 
 
 int
 main(int argc, char *argv[])
 {
+       struct manconf   conf;
        struct curparse  curp;
        struct mansearch search;
-       struct manpaths  paths;
        char            *auxpaths;
        char            *defos;
        unsigned char   *uc;
        struct manpage  *res, *resp;
        char            *conf_file, *defpaths;
        size_t           isec, i, sz;
-       int              prio, best_prio, synopsis_only;
+       int              prio, best_prio;
        char             sec;
-       enum mandoclevel rc, rctmp;
+       enum mandoclevel rctmp;
        enum outmode     outmode;
        int              fd;
        int              show_usage;
-       int              use_pager;
        int              options;
        int              c;
+       pid_t            pager_pid;  /* 0: don't use; 1: not yet spawned. */
 
        if (argc < 1)
                progname = "mandoc";
@@ -148,7 +152,7 @@ main(int argc, char *argv[])
 
        /* Search options. */
 
-       memset(&paths, 0, sizeof(struct manpaths));
+       memset(&conf, 0, sizeof(conf));
        conf_file = defpaths = NULL;
        auxpaths = NULL;
 
@@ -171,12 +175,12 @@ main(int argc, char *argv[])
        memset(&curp, 0, sizeof(struct curparse));
        curp.outtype = OUTT_LOCALE;
        curp.wlevel  = MANDOCLEVEL_BADARG;
+       curp.outopts = &conf.output;
        options = MPARSE_SO | MPARSE_UTF8 | MPARSE_LATIN1;
        defos = NULL;
 
-       use_pager = 1;
+       pager_pid = 1;
        show_usage = 0;
-       synopsis_only = 0;
        outmode = OUTMODE_DEF;
 
        while (-1 != (c = getopt(argc, argv,
@@ -189,15 +193,14 @@ main(int argc, char *argv[])
                        conf_file = optarg;
                        break;
                case 'c':
-                       use_pager = 0;
+                       pager_pid = 0;
                        break;
                case 'f':
                        search.argmode = ARG_WORD;
                        break;
                case 'h':
-                       (void)strlcat(curp.outopts, "synopsis,", BUFSIZ);
-                       synopsis_only = 1;
-                       use_pager = 0;
+                       conf.output.synopsisonly = 1;
+                       pager_pid = 0;
                        outmode = OUTMODE_ALL;
                        break;
                case 'I':
@@ -237,8 +240,9 @@ main(int argc, char *argv[])
                        break;
                case 'O':
                        search.outkey = optarg;
-                       (void)strlcat(curp.outopts, optarg, BUFSIZ);
-                       (void)strlcat(curp.outopts, ",", BUFSIZ);
+                       while (optarg != NULL)
+                               manconf_output(&conf.output,
+                                   strsep(&optarg, ","));
                        break;
                case 'S':
                        search.arch = optarg;
@@ -272,7 +276,7 @@ main(int argc, char *argv[])
                switch (search.argmode) {
                case ARG_FILE:
                        outmode = OUTMODE_ALL;
-                       use_pager = 0;
+                       pager_pid = 0;
                        break;
                case ARG_NAME:
                        outmode = OUTMODE_ONE;
@@ -333,10 +337,11 @@ main(int argc, char *argv[])
 
                /* Access the mandoc database. */
 
-               manpath_parse(&paths, conf_file, defpaths, auxpaths);
+               manconf_parse(&conf, conf_file, defpaths, auxpaths);
 #if HAVE_SQLITE3
                mansearch_setup(1);
-               if( ! mansearch(&search, &paths, argc, argv, &res, &sz))
+               if ( ! mansearch(&search, &conf.manpath,
+                   argc, argv, &res, &sz))
                        usage(search.argmode);
 #else
                if (search.argmode != ARG_NAME) {
@@ -348,7 +353,8 @@ main(int argc, char *argv[])
 #endif
 
                if (sz == 0 && search.argmode == ARG_NAME)
-                       fs_search(&search, &paths, argc, argv, &res, &sz);
+                       fs_search(&search, &conf.manpath,
+                           argc, argv, &res, &sz);
 
                if (sz == 0) {
                        rc = MANDOCLEVEL_BADARG;
@@ -417,9 +423,9 @@ main(int argc, char *argv[])
                mparse_keep(curp.mp);
 
        if (argc < 1) {
-               if (use_pager && isatty(STDOUT_FILENO))
-                       spawn_pager();
-               parse(&curp, STDIN_FILENO, "<stdin>", &rc);
+               if (pager_pid == 1 && isatty(STDOUT_FILENO))
+                       pager_pid = spawn_pager();
+               parse(&curp, STDIN_FILENO, "<stdin>");
        }
 
        while (argc > 0) {
@@ -429,22 +435,18 @@ main(int argc, char *argv[])
                        rc = rctmp;
 
                if (fd != -1) {
-                       if (use_pager && isatty(STDOUT_FILENO))
-                               spawn_pager();
-                       use_pager = 0;
+                       if (pager_pid == 1 && isatty(STDOUT_FILENO))
+                               pager_pid = spawn_pager();
 
                        if (resp == NULL)
-                               parse(&curp, fd, *argv, &rc);
+                               parse(&curp, fd, *argv);
                        else if (resp->form & FORM_SRC) {
                                /* For .so only; ignore failure. */
-                               chdir(paths.paths[resp->ipath]);
-                               parse(&curp, fd, resp->file, &rc);
-                       } else {
-                               rctmp = passthrough(resp->file, fd,
-                                   synopsis_only);
-                               if (rc < rctmp)
-                                       rc = rctmp;
-                       }
+                               chdir(conf.manpath.paths[resp->ipath]);
+                               parse(&curp, fd, resp->file);
+                       } else
+                               passthrough(resp->file, fd,
+                                   conf.output.synopsisonly);
 
                        rctmp = mparse_wait(curp.mp);
                        if (rc < rctmp)
@@ -472,7 +474,7 @@ main(int argc, char *argv[])
 
 out:
        if (search.argmode != ARG_FILE) {
-               manpath_free(&paths);
+               manconf_free(&conf);
 #if HAVE_SQLITE3
                mansearch_free(res, sz);
                mansearch_setup(0);
@@ -481,6 +483,17 @@ out:
 
        free(defos);
 
+       /*
+        * If a pager is attached, flush the pipe leading to it
+        * and signal end of file such that the user can browse
+        * to the end.  Then wait for the user to close the pager.
+        */
+
+       if (pager_pid != 0 && pager_pid != 1) {
+               fclose(stdout);
+               waitpid(pager_pid, NULL, 0);
+       }
+
        return((int)rc);
 }
 
@@ -490,9 +503,9 @@ usage(enum argmode argmode)
 
        switch (argmode) {
        case ARG_FILE:
-               fputs("usage: mandoc [-acfhkl] [-Ios=name] "
-                   "[-Kencoding] [-mformat] [-Ooption]\n"
-                   "\t      [-Toutput] [-Wlevel] [file ...]\n", stderr);
+               fputs("usage: mandoc [-acfhkl] [-I os=name] "
+                   "[-K encoding] [-mformat] [-O option]\n"
+                   "\t      [-T output] [-W level] [file ...]\n", stderr);
                break;
        case ARG_NAME:
                fputs("usage: man [-acfhklw] [-C file] [-I os=name] "
@@ -618,10 +631,9 @@ fs_search(const struct mansearch *cfg, const struct manpaths *paths,
 }
 
 static void
-parse(struct curparse *curp, int fd, const char *file,
-       enum mandoclevel *level)
+parse(struct curparse *curp, int fd, const char *file)
 {
-       enum mandoclevel  rc;
+       enum mandoclevel  rctmp;
        struct mdoc      *mdoc;
        struct man       *man;
 
@@ -630,15 +642,17 @@ parse(struct curparse *curp, int fd, const char *file,
        assert(file);
        assert(fd >= -1);
 
-       rc = mparse_readfd(curp->mp, fd, file);
+       rctmp = mparse_readfd(curp->mp, fd, file);
+       if (rc < rctmp)
+               rc = rctmp;
 
        /*
         * With -Wstop and warnings or errors of at least the requested
         * level, do not produce output.
         */
 
-       if (MANDOCLEVEL_OK != rc && curp->wstop)
-               goto cleanup;
+       if (rctmp != MANDOCLEVEL_OK && curp->wstop)
+               return;
 
        /* If unset, allocate output dev now (if applicable). */
 
@@ -716,13 +730,9 @@ parse(struct curparse *curp, int fd, const char *file,
                (*curp->outman)(curp->outdata, man);
        if (mdoc && curp->outmdoc)
                (*curp->outmdoc)(curp->outdata, mdoc);
-
-cleanup:
-       if (*level < rc)
-               *level = rc;
 }
 
-static enum mandoclevel
+static void
 passthrough(const char *file, int fd, int synopsis_only)
 {
        const char       synb[] = "S\bSY\bYN\bNO\bOP\bPS\bSI\bIS\bS";
@@ -780,12 +790,13 @@ passthrough(const char *file, int fd, int synopsis_only)
 
 done:
        fclose(stream);
-       return(MANDOCLEVEL_OK);
+       return;
 
 fail:
        fprintf(stderr, "%s: %s: SYSERR: %s: %s",
            progname, file, syscall, strerror(errno));
-       return(MANDOCLEVEL_SYSERR);
+       if (rc < MANDOCLEVEL_SYSERR)
+               rc = MANDOCLEVEL_SYSERR;
 }
 
 static int
@@ -930,6 +941,13 @@ mmsg(enum mandocerr t, enum mandoclevel lvl,
 }
 
 static void
+handle_sigpipe(int signum)
+{
+
+       exit((int)rc);
+}
+
+static pid_t
 spawn_pager(void)
 {
 #define MAX_PAGER_ARGS 16
@@ -938,31 +956,34 @@ spawn_pager(void)
        char            *cp;
        int              fildes[2];
        int              argc;
+       pid_t            pager_pid;
 
        if (pipe(fildes) == -1) {
                fprintf(stderr, "%s: pipe: %s\n",
                    progname, strerror(errno));
-               return;
+               return(0);
        }
 
-       switch (fork()) {
+       switch (pager_pid = fork()) {
        case -1:
                fprintf(stderr, "%s: fork: %s\n",
                    progname, strerror(errno));
                exit((int)MANDOCLEVEL_SYSERR);
        case 0:
+               break;
+       default:
                close(fildes[0]);
                if (dup2(fildes[1], STDOUT_FILENO) == -1) {
                        fprintf(stderr, "%s: dup output: %s\n",
                            progname, strerror(errno));
                        exit((int)MANDOCLEVEL_SYSERR);
                }
-               return;
-       default:
-               break;
+               close(fildes[1]);
+               signal(SIGPIPE, handle_sigpipe);
+               return(pager_pid);
        }
 
-       /* The original process becomes the pager. */
+       /* The child process becomes the pager. */
 
        close(fildes[1]);
        if (dup2(fildes[0], STDIN_FILENO) == -1) {
@@ -970,12 +991,13 @@ spawn_pager(void)
                    progname, strerror(errno));
                exit((int)MANDOCLEVEL_SYSERR);
        }
+       close(fildes[0]);
 
        pager = getenv("MANPAGER");
        if (pager == NULL || *pager == '\0')
                pager = getenv("PAGER");
        if (pager == NULL || *pager == '\0')
-               pager = "/usr/bin/more -s";
+               pager = "more -s";
        cp = mandoc_strdup(pager);
 
        /*