]> git.cameronkatri.com Git - mandoc.git/blobdiff - mansearch.c
modernize style: "return" is not a function
[mandoc.git] / mansearch.c
index 05e2a0a2b3c638ca53eec4219990a0f02885b82c..e019b4877f0eeb2e3c55194c7f5a4570814bb483 100644 (file)
@@ -1,4 +1,4 @@
-/*     $Id: mansearch.c,v 1.55 2015/03/11 13:11:22 schwarze Exp $ */
+/*     $Id: mansearch.c,v 1.58 2015/10/06 18:32:19 schwarze Exp $ */
 /*
  * Copyright (c) 2012 Kristaps Dzonsons <kristaps@bsd.lv>
  * Copyright (c) 2013, 2014, 2015 Ingo Schwarze <schwarze@openbsd.org>
@@ -7,9 +7,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
@@ -46,7 +46,7 @@
 
 #include "mandoc.h"
 #include "mandoc_aux.h"
-#include "manpath.h"
+#include "manconf.h"
 #include "mansearch.h"
 
 extern int mansearch_keymax;
@@ -121,7 +121,7 @@ mansearch_setup(int start)
        if (start) {
                if (NULL != pagecache) {
                        fprintf(stderr, "pagecache already enabled\n");
-                       return((int)MANDOCLEVEL_BADARG);
+                       return (int)MANDOCLEVEL_BADARG;
                }
 
                pagecache = mmap(NULL, PC_PAGESIZE * PC_NUMPAGES,
@@ -131,30 +131,30 @@ mansearch_setup(int start)
                if (MAP_FAILED == pagecache) {
                        perror("mmap");
                        pagecache = NULL;
-                       return((int)MANDOCLEVEL_SYSERR);
+                       return (int)MANDOCLEVEL_SYSERR;
                }
 
                c = sqlite3_config(SQLITE_CONFIG_PAGECACHE,
                    pagecache, PC_PAGESIZE, PC_NUMPAGES);
 
                if (SQLITE_OK == c)
-                       return((int)MANDOCLEVEL_OK);
+                       return (int)MANDOCLEVEL_OK;
 
                fprintf(stderr, "pagecache: %s\n", sqlite3_errstr(c));
 
        } else if (NULL == pagecache) {
                fprintf(stderr, "pagecache missing\n");
-               return((int)MANDOCLEVEL_BADARG);
+               return (int)MANDOCLEVEL_BADARG;
        }
 
        if (-1 == munmap(pagecache, PC_PAGESIZE * PC_NUMPAGES)) {
                perror("munmap");
                pagecache = NULL;
-               return((int)MANDOCLEVEL_SYSERR);
+               return (int)MANDOCLEVEL_SYSERR;
        }
 
        pagecache = NULL;
-       return((int)MANDOCLEVEL_OK);
+       return (int)MANDOCLEVEL_OK;
 }
 
 int
@@ -163,7 +163,6 @@ mansearch(const struct mansearch *search,
                int argc, char *argv[],
                struct manpage **res, size_t *sz)
 {
-       int              fd, rc, c, indexbit;
        int64_t          pageid;
        uint64_t         outbit, iterbit;
        char             buf[PATH_MAX];
@@ -177,23 +176,20 @@ mansearch(const struct mansearch *search,
        struct ohash     htab;
        unsigned int     idx;
        size_t           i, j, cur, maxres;
+       int              c, chdir_status, getcwd_status, indexbit;
+
+       if (argc == 0 || (e = exprcomp(search, argc, argv)) == NULL) {
+               *sz = 0;
+               return 0;
+       }
 
        info.calloc = hash_calloc;
        info.alloc = hash_alloc;
        info.free = hash_free;
        info.key_offset = offsetof(struct match, pageid);
 
-       *sz = cur = maxres = 0;
-       sql = NULL;
+       cur = maxres = 0;
        *res = NULL;
-       fd = -1;
-       e = NULL;
-       rc = 0;
-
-       if (0 == argc)
-               goto out;
-       if (NULL == (e = exprcomp(search, argc, argv)))
-               goto out;
 
        if (NULL != search->outkey) {
                outbit = TYPE_Nd;
@@ -210,19 +206,18 @@ mansearch(const struct mansearch *search,
                outbit = 0;
 
        /*
-        * Save a descriptor to the current working directory.
-        * Since pathnames in the "paths" variable might be relative,
-        * and we'll be chdir()ing into them, we need to keep a handle
-        * on our current directory from which to start the chdir().
+        * Remember the original working directory, if possible.
+        * This will be needed if the second or a later directory
+        * is given as a relative path.
+        * Do not error out if the current directory is not
+        * searchable: Maybe it won't be needed after all.
         */
 
-       if (NULL == getcwd(buf, PATH_MAX)) {
-               perror("getcwd");
-               goto out;
-       } else if (-1 == (fd = open(buf, O_RDONLY, 0))) {
-               perror(buf);
-               goto out;
-       }
+       if (getcwd(buf, PATH_MAX) == NULL) {
+               getcwd_status = 0;
+               (void)strlcpy(buf, strerror(errno), sizeof(buf));
+       } else
+               getcwd_status = 1;
 
        sql = sql_statement(e);
 
@@ -234,15 +229,23 @@ mansearch(const struct mansearch *search,
         * scan it for our match expression.
         */
 
+       chdir_status = 0;
        for (i = 0; i < paths->sz; i++) {
-               if (-1 == fchdir(fd)) {
-                       perror(buf);
-                       free(*res);
-                       break;
-               } else if (-1 == chdir(paths->paths[i])) {
+               if (chdir_status && paths->paths[i][0] != '/') {
+                       if ( ! getcwd_status) {
+                               fprintf(stderr, "%s: getcwd: %s\n",
+                                   paths->paths[i], buf);
+                               continue;
+                       } else if (chdir(buf) == -1) {
+                               perror(buf);
+                               continue;
+                       }
+               }
+               if (chdir(paths->paths[i]) == -1) {
                        perror(paths->paths[i]);
                        continue;
                }
+               chdir_status = 1;
 
                c = sqlite3_open_v2(MANDOC_DB, &db,
                    SQLITE_OPEN_READONLY, NULL);
@@ -370,17 +373,12 @@ mansearch(const struct mansearch *search,
                        break;
        }
        qsort(*res, cur, sizeof(struct manpage), manpage_compare);
-       rc = 1;
-out:
-       if (-1 != fd) {
-               if (-1 == fchdir(fd))
-                       perror(buf);
-               close(fd);
-       }
+       if (chdir_status && getcwd_status && chdir(buf) == -1)
+               perror(buf);
        exprfree(e);
        free(sql);
        *sz = cur;
-       return(rc);
+       return 1;
 }
 
 void
@@ -404,9 +402,9 @@ manpage_compare(const void *vp1, const void *vp2)
 
        mp1 = vp1;
        mp2 = vp2;
-       return( (diff = mp2->bits - mp1->bits) ? diff :
-               (diff = mp1->sec - mp2->sec) ? diff :
-               strcasecmp(mp1->names, mp2->names));
+       return (diff = mp2->bits - mp1->bits) ? diff :
+           (diff = mp1->sec - mp2->sec) ? diff :
+           strcasecmp(mp1->names, mp2->names);
 }
 
 static void
@@ -567,7 +565,7 @@ buildoutput(sqlite3 *db, sqlite3_stmt *s, uint64_t pageid, uint64_t outbit)
        if (SQLITE_DONE != c)
                fprintf(stderr, "%s\n", sqlite3_errmsg(db));
        sqlite3_reset(s);
-       return(output);
+       return output;
 }
 
 /*
@@ -662,7 +660,7 @@ sql_statement(const struct expr *e)
                needop = 1;
        }
 
-       return(sql);
+       return sql;
 }
 
 /*
@@ -745,12 +743,12 @@ exprcomp(const struct mansearch *search, int argc, char *argv[])
                toopen = logic = igncase = 0;
        }
        if ( ! (toopen || logic || igncase || toclose))
-               return(first);
+               return first;
 
 fail:
        if (NULL != first)
                exprfree(first);
-       return(NULL);
+       return NULL;
 }
 
 static struct expr *
@@ -763,7 +761,7 @@ exprterm(const struct mansearch *search, char *buf, int cs)
        int              i, irc;
 
        if ('\0' == *buf)
-               return(NULL);
+               return NULL;
 
        e = mandoc_calloc(1, sizeof(struct expr));
 
@@ -771,7 +769,7 @@ exprterm(const struct mansearch *search, char *buf, int cs)
                e->bits = TYPE_Nm;
                e->substr = buf;
                e->equal = 1;
-               return(e);
+               return e;
        }
 
        /*
@@ -809,12 +807,12 @@ exprterm(const struct mansearch *search, char *buf, int cs)
                        regerror(irc, &e->regexp, errbuf, sizeof(errbuf));
                        fprintf(stderr, "regcomp: %s\n", errbuf);
                        free(e);
-                       return(NULL);
+                       return NULL;
                }
        }
 
        if (e->bits)
-               return(e);
+               return e;
 
        /*
         * Parse out all possible fields.
@@ -836,13 +834,13 @@ exprterm(const struct mansearch *search, char *buf, int cs)
                if (i == mansearch_keymax) {
                        if (strcasecmp(key, "any")) {
                                free(e);
-                               return(NULL);
+                               return NULL;
                        }
                        e->bits |= ~0ULL;
                }
        }
 
-       return(e);
+       return e;
 }
 
 static void
@@ -861,14 +859,14 @@ static void *
 hash_calloc(size_t nmemb, size_t sz, void *arg)
 {
 
-       return(mandoc_calloc(nmemb, sz));
+       return mandoc_calloc(nmemb, sz);
 }
 
 static void *
 hash_alloc(size_t sz, void *arg)
 {
 
-       return(mandoc_malloc(sz));
+       return mandoc_malloc(sz);
 }
 
 static void