Use nested namespace and GC unused code.

5 files changed, 10 insertions, 66 deletions

diff --git a/searchlib/src/vespa/searchlib/bitcompression/compression.cpp b/searchlib/src/vespa/searchlib/bitcompression/compression.cpp
index 1031aab71f5..2aa21f9871d 100644
--- a/searchlib/src/vespa/searchlib/bitcompression/compression.cpp
+++ b/searchlib/src/vespa/searchlib/bitcompression/compression.cpp
@@ -449,5 +449,4 @@ template class FeatureDecodeContext<false>;
 template class FeatureEncodeContext<true>;
 template class FeatureEncodeContext<false>;
 
-
 }
diff --git a/searchlib/src/vespa/searchlib/bitcompression/compression.h b/searchlib/src/vespa/searchlib/bitcompression/compression.h
index 931e4165b21..482a9d8b6bb 100644
--- a/searchlib/src/vespa/searchlib/bitcompression/compression.h
+++ b/searchlib/src/vespa/searchlib/bitcompression/compression.h
@@ -7,19 +7,13 @@
 #include <vespa/vespalib/stllike/string.h>
 #include <cassert>
 
-namespace vespalib {
+namespace vespalib { class GenericHeader; }
 
-class GenericHeader;
+namespace search::index { class DocIdAndFeatures; }
 
-}
-
-namespace search {
-
-  namespace index { class DocIdAndFeatures; }
-
-namespace fef { class TermFieldMatchDataArray; }
+namespace search::fef { class TermFieldMatchDataArray; }
 
-namespace bitcompression {
+namespace search::bitcompression {
 
 class Position {
 public:
@@ -1733,7 +1727,4 @@ extern template class FeatureDecodeContext<false>;
 extern template class FeatureEncodeContext<true>;
 extern template class FeatureEncodeContext<false>;
 
-} // namespace bitcompression
-
-} // namespace search
-
+}
diff --git a/searchlib/src/vespa/searchlib/bitcompression/countcompression.cpp b/searchlib/src/vespa/searchlib/bitcompression/countcompression.cpp
index df699b792bc..642c5acdae5 100644
--- a/searchlib/src/vespa/searchlib/bitcompression/countcompression.cpp
+++ b/searchlib/src/vespa/searchlib/bitcompression/countcompression.cpp
@@ -9,7 +9,6 @@ namespace search::bitcompression {
 #define K_VALUE_COUNTFILE_LASTDOCID 22
 #define K_VALUE_COUNTFILE_NUMCHUNKS 1
 #define K_VALUE_COUNTFILE_CHUNKNUMDOCS 18
-#define K_VALUE_COUNTFILE_WORDNUMDELTA 0
 #define K_VALUE_COUNTFILE_SPNUMDOCS 0
 
 
@@ -89,27 +88,6 @@ readCounts(PostingListCounts &counts)
         _readContext->readComprBuffer();
 }
 
-
-void
-PostingListCountFileDecodeContext::
-readWordNum(uint64_t &wordNum)
-{
-    UC64_DECODECONTEXT_CONSTRUCTOR(o, _);
-    uint32_t length;
-    uint64_t val64;
-    const uint64_t *valE = _valE;
-
-    UC64BE_DECODEEXPGOLOMB_NS(o,
-                              K_VALUE_COUNTFILE_WORDNUMDELTA,
-                              EC);
-    wordNum = _minWordNum + val64;
-    UC64_DECODECONTEXT_STORE(o, _);
-    if (__builtin_expect(oCompr >= valE, false))
-        _readContext->readComprBuffer();
-    _minWordNum = wordNum + 1;
-}
-
-
 void
 PostingListCountFileDecodeContext::
 copyParams(const PostingListCountFileDecodeContext &rhs)
@@ -164,21 +142,6 @@ writeCounts(const PostingListCounts &counts)
         _writeContext->writeComprBuffer(false);
 }
 
-
-void
-PostingListCountFileEncodeContext::
-writeWordNum(uint64_t wordNum)
-{
-    assert(wordNum >= _minWordNum);
-    assert(wordNum <= _numWordIds);
-    encodeExpGolomb(wordNum - _minWordNum,
-                    K_VALUE_COUNTFILE_WORDNUMDELTA);
-    if (__builtin_expect(_valI >= _valE, false))
-        _writeContext->writeComprBuffer(false);
-    _minWordNum = wordNum + 1;
-}
-
-
 void
 PostingListCountFileEncodeContext::
 copyParams(const PostingListCountFileEncodeContext &rhs)
@@ -189,5 +152,4 @@ copyParams(const PostingListCountFileEncodeContext &rhs)
     _numWordIds = rhs._numWordIds;
 }
 
-
 }
diff --git a/searchlib/src/vespa/searchlib/bitcompression/countcompression.h b/searchlib/src/vespa/searchlib/bitcompression/countcompression.h
index df007d3c3fd..06e01c4c6f4 100644
--- a/searchlib/src/vespa/searchlib/bitcompression/countcompression.h
+++ b/searchlib/src/vespa/searchlib/bitcompression/countcompression.h
@@ -32,7 +32,6 @@ public:
     }
 
     void readCounts(PostingListCounts &counts);
-    void readWordNum(uint64_t &wordNum);
 
     static uint64_t noWordNum() {
         return std::numeric_limits<uint64_t>::max();
@@ -64,7 +63,6 @@ public:
     }
 
     void writeCounts(const PostingListCounts &counts);
-    void writeWordNum(uint64_t wordNum);
 
     static uint64_t noWordNum() {
         return std::numeric_limits<uint64_t>::max();
diff --git a/searchlib/src/vespa/searchlib/diskindex/zcposting.h b/searchlib/src/vespa/searchlib/diskindex/zcposting.h
index 9a650abdd40..8c69a051e83 100644
--- a/searchlib/src/vespa/searchlib/diskindex/zcposting.h
+++ b/searchlib/src/vespa/searchlib/diskindex/zcposting.h
@@ -7,16 +7,12 @@
 #include <vespa/searchlib/bitcompression/compression.h>
 #include <vespa/fastos/file.h>
 
-namespace search {
-
-namespace index {
-
-class PostingListCountFileSeqRead;
-class PostingListCountFileSeqWrite;
-
+namespace search::index {
+    class PostingListCountFileSeqRead;
+    class PostingListCountFileSeqWrite;
 }
 
-namespace diskindex {
+namespace search::diskindex {
 
 class Zc4PostingSeqRead : public index::PostingListFileSeqRead
 {
@@ -230,6 +226,4 @@ public:
     void flushWordNoSkip() override;
 };
 
-} // namespace diskindex
-
-} // namespace search
+}