diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2022-01-26 19:20:01 +0000 |
---|---|---|
committer | Henning Baldersheim <balder@yahoo-inc.com> | 2022-01-26 19:20:01 +0000 |
commit | 3a0ab5a2d6ac2b7d94a2726862e4cd722c7c34a1 (patch) | |
tree | 49005016f300ec4fd294c296c92d1095a5cb6659 /searchlib | |
parent | 70583db573e8e0033694e8c3d403d256f9e8cfed (diff) |
Add [[nodiscard]] to Read() too
Diffstat (limited to 'searchlib')
-rw-r--r-- | searchlib/src/vespa/searchlib/diskindex/bitvectordictionary.cpp | 3 | ||||
-rw-r--r-- | searchlib/src/vespa/searchlib/diskindex/wordnummapper.cpp | 5 |
2 files changed, 5 insertions, 3 deletions
diff --git a/searchlib/src/vespa/searchlib/diskindex/bitvectordictionary.cpp b/searchlib/src/vespa/searchlib/diskindex/bitvectordictionary.cpp index 29c82936076..f6dd5a318ae 100644 --- a/searchlib/src/vespa/searchlib/diskindex/bitvectordictionary.cpp +++ b/searchlib/src/vespa/searchlib/diskindex/bitvectordictionary.cpp @@ -53,7 +53,8 @@ BitVectorDictionary::open(const vespalib::string &pathPrefix, size_t bufSize = sizeof(WordSingleKey) * numEntries; assert(idxFile.GetSize() >= static_cast<int64_t>(idxHeaderLen + bufSize)); if (bufSize > 0) { - idxFile.Read(&_entries[0], bufSize); + ssize_t has_read = idxFile.Read(&_entries[0], bufSize); + assert(has_read == ssize_t(bufSize)); } } diff --git a/searchlib/src/vespa/searchlib/diskindex/wordnummapper.cpp b/searchlib/src/vespa/searchlib/diskindex/wordnummapper.cpp index cd9cd6c34da..3c74ab5ff07 100644 --- a/searchlib/src/vespa/searchlib/diskindex/wordnummapper.cpp +++ b/searchlib/src/vespa/searchlib/diskindex/wordnummapper.cpp @@ -2,6 +2,7 @@ #include "wordnummapper.h" #include <vespa/fastlib/io/bufferedfile.h> +#include <cassert> namespace search::diskindex { @@ -30,8 +31,8 @@ WordNumMapping::readMappingFile(const vespalib::string &name, map.resize(tempfileentries + 2); _oldDictSize = tempfileentries; - old2newwordfile.Read(&map[1], - static_cast<size_t>(tempfilesize)); + ssize_t has_read = old2newwordfile.Read(&map[1], static_cast<size_t>(tempfilesize)); + assert(has_read == tempfilesize); map[0] = noWordNum(); map[tempfileentries + 1] = noWordNumHigh(); } |