Take an assert off the hot path.

[plocate] / plocate.cpp

diff --git a/plocate.cpp b/plocate.cpp
index 423aaf7e632f408bcdedcb48163c1918e7a7510b..c47c6480465a7f9e70472e7cbac984e3c6ddc78d 100644 (file)
--- a/plocate.cpp
+++ b/plocate.cpp
@@ -1,4 +1,5 @@
 #include "access_rx_cache.h"
+#include "complete_pread.h"
 #include "db.h"
 #include "dprintf.h"
 #include "io_uring_engine.h"
@@ -42,9 +43,7 @@
 using namespace std;
 using namespace std::chrono;
 
-#define DEFAULT_DBPATH "/var/lib/mlocate/plocate.db"
-
-const char *dbpath = DEFAULT_DBPATH;
+const char *dbpath = DBFILE;
 bool ignore_case = false;
 bool only_count = false;
 bool print_nul = false;
@@ -106,6 +105,10 @@ Corpus::Corpus(int fd, IOUringEngine *engine)
                hdr.zstd_dictionary_offset_bytes = 0;
                hdr.zstd_dictionary_length_bytes = 0;
        }
+       if (hdr.max_version < 2) {
+               // This too. (We ignore the other max_version 2 fields.)
+               hdr.check_visibility = true;
+       }
 }
 
 Corpus::~Corpus()
@@ -231,7 +234,7 @@ void scan_file_block(const vector<Needle> &needles, string_view compressed,
 size_t scan_docids(const vector<Needle> &needles, const vector<uint32_t> &docids, const Corpus &corpus, IOUringEngine *engine)
 {
        Serializer docids_in_order;
-       AccessRXCache access_rx_cache(engine);
+       AccessRXCache access_rx_cache(engine, corpus.get_hdr().check_visibility);
        atomic<uint64_t> matched{ 0 };
        for (size_t i = 0; i < docids.size(); ++i) {
                uint32_t docid = docids[i];
@@ -307,7 +310,7 @@ uint64_t scan_all_docids(const vector<Needle> &needles, int fd, const Corpus &co
                }
        }
 
-       AccessRXCache access_rx_cache(nullptr);
+       AccessRXCache access_rx_cache(nullptr, corpus.get_hdr().check_visibility);
        Serializer serializer;
        uint32_t num_blocks = corpus.get_num_filename_blocks();
        unique_ptr<uint64_t[]> offsets(new uint64_t[num_blocks + 1]);
@@ -655,7 +658,7 @@ void usage()
                "  -b, --basename         search only the file name portion of path names\n"
                "  -c, --count            print number of matches instead of the matches\n"
                "  -d, --database DBPATH  search for files in DBPATH\n"
-               "                         (default is " DEFAULT_DBPATH ")\n"
+               "                         (default is " DBFILE ")\n"
                "  -i, --ignore-case      search case-insensitively\n"
                "  -l, --limit LIMIT      stop after LIMIT matches\n"
                "  -0, --null             delimit matches by NUL instead of newline\n"
@@ -668,7 +671,7 @@ void usage()
 
 void version()
 {
-       printf("plocate %s\n", PLOCATE_VERSION);
+       printf("%s %s\n", PACKAGE_NAME, PACKAGE_VERSION);
        printf("Copyright 2020 Steinar H. Gunderson\n");
        printf("License GPLv2+: GNU GPL version 2 or later <https://gnu.org/licenses/gpl.html>.\n");
        printf("This is free software: you are free to change and redistribute it.\n");