From a3e571660a656fbed65c1b3f57a6b9b4f8a57f04 Mon Sep 17 00:00:00 2001 From: Henning Baldersheim Date: Tue, 15 Aug 2017 13:47:57 +0200 Subject: Use nested namespace --- .../src/vespa/searchlib/attribute/reference_attribute.cpp | 6 ++---- searchlib/src/vespa/searchlib/attribute/reference_attribute.h | 7 ++----- searchlib/src/vespa/searchlib/datastore/allocator.h | 4 +--- searchlib/src/vespa/searchlib/datastore/allocator.hpp | 5 ++--- searchlib/src/vespa/searchlib/datastore/array_store.h | 4 +--- searchlib/src/vespa/searchlib/datastore/array_store.hpp | 5 ++--- searchlib/src/vespa/searchlib/datastore/array_store_config.h | 4 +--- searchlib/src/vespa/searchlib/datastore/buffer_type.cpp | 6 ++---- searchlib/src/vespa/searchlib/datastore/buffer_type.h | 7 ++----- searchlib/src/vespa/searchlib/datastore/bufferstate.cpp | 6 ++---- searchlib/src/vespa/searchlib/datastore/bufferstate.h | 8 ++------ searchlib/src/vespa/searchlib/datastore/datastore.cpp | 6 ++---- searchlib/src/vespa/searchlib/datastore/datastore.h | 11 ++++------- searchlib/src/vespa/searchlib/datastore/datastore.hpp | 6 ++---- searchlib/src/vespa/searchlib/datastore/datastorebase.cpp | 7 +++---- searchlib/src/vespa/searchlib/datastore/datastorebase.h | 7 ++----- searchlib/src/vespa/searchlib/datastore/entryref.h | 9 +++------ searchlib/src/vespa/searchlib/datastore/free_list_allocator.h | 4 +--- .../src/vespa/searchlib/datastore/free_list_allocator.hpp | 5 ++--- searchlib/src/vespa/searchlib/datastore/handle.h | 4 +--- .../src/vespa/searchlib/datastore/i_compaction_context.h | 4 +--- searchlib/src/vespa/searchlib/datastore/raw_allocator.h | 4 +--- searchlib/src/vespa/searchlib/datastore/raw_allocator.hpp | 5 ++--- searchlib/src/vespa/searchlib/datastore/unique_store.h | 4 +--- searchlib/src/vespa/searchlib/datastore/unique_store.hpp | 4 +--- .../src/vespa/searchlib/datastore/unique_store_builder.h | 4 +--- .../src/vespa/searchlib/datastore/unique_store_builder.hpp | 5 ++--- searchlib/src/vespa/searchlib/datastore/unique_store_saver.h | 4 +--- .../src/vespa/searchlib/datastore/unique_store_saver.hpp | 4 +--- 29 files changed, 50 insertions(+), 109 deletions(-) (limited to 'searchlib') diff --git a/searchlib/src/vespa/searchlib/attribute/reference_attribute.cpp b/searchlib/src/vespa/searchlib/attribute/reference_attribute.cpp index 9f5d9cacd55..d85c0e0bb2c 100644 --- a/searchlib/src/vespa/searchlib/attribute/reference_attribute.cpp +++ b/searchlib/src/vespa/searchlib/attribute/reference_attribute.cpp @@ -11,8 +11,7 @@ #include #include -namespace search { -namespace attribute { +namespace search::attribute { namespace { @@ -306,7 +305,7 @@ IMPLEMENT_IDENTIFIABLE_ABSTRACT(ReferenceAttribute, AttributeVector); } -namespace datastore { +namespace search::datastore { using Reference = attribute::ReferenceAttribute::Reference; @@ -315,4 +314,3 @@ template class UniqueStoreBuilder>; template class UniqueStoreSaver>; } -} diff --git a/searchlib/src/vespa/searchlib/attribute/reference_attribute.h b/searchlib/src/vespa/searchlib/attribute/reference_attribute.h index 0df40cf8b2a..1ea0cc0e883 100644 --- a/searchlib/src/vespa/searchlib/attribute/reference_attribute.h +++ b/searchlib/src/vespa/searchlib/attribute/reference_attribute.h @@ -7,11 +7,9 @@ #include #include -namespace search { +namespace search { class IGidToLidMapperFactory; } -class IGidToLidMapperFactory; - -namespace attribute { +namespace search::attribute { /* * Attribute vector which maintains a lid-2-lid mapping from local document ids to global ids (referencing external documents) @@ -89,4 +87,3 @@ public: }; } -} diff --git a/searchlib/src/vespa/searchlib/datastore/allocator.h b/searchlib/src/vespa/searchlib/datastore/allocator.h index 8a1579771d5..8a522266c1a 100644 --- a/searchlib/src/vespa/searchlib/datastore/allocator.h +++ b/searchlib/src/vespa/searchlib/datastore/allocator.h @@ -7,8 +7,7 @@ #include "handle.h" #include -namespace search { -namespace datastore { +namespace search::datastore { /** * Allocator used to allocate entries of a specific type in an underlying data store. @@ -35,4 +34,3 @@ public: }; } -} diff --git a/searchlib/src/vespa/searchlib/datastore/allocator.hpp b/searchlib/src/vespa/searchlib/datastore/allocator.hpp index b36ae131cc8..9c4f8568924 100644 --- a/searchlib/src/vespa/searchlib/datastore/allocator.hpp +++ b/searchlib/src/vespa/searchlib/datastore/allocator.hpp @@ -5,8 +5,7 @@ #include "allocator.h" #include "bufferstate.h" -namespace search { -namespace datastore { +namespace search::datastore { template Allocator::Allocator(DataStoreBase &store, uint32_t typeId) @@ -70,4 +69,4 @@ Allocator::allocArray(size_t size) } } -} + diff --git a/searchlib/src/vespa/searchlib/datastore/array_store.h b/searchlib/src/vespa/searchlib/datastore/array_store.h index 5ff4be682ec..c6d3fff1b85 100644 --- a/searchlib/src/vespa/searchlib/datastore/array_store.h +++ b/searchlib/src/vespa/searchlib/datastore/array_store.h @@ -10,8 +10,7 @@ #include "i_compaction_context.h" #include -namespace search { -namespace datastore { +namespace search::datastore { /** * Datastore for storing arrays of type EntryT that is accessed via a 32-bit EntryRef. @@ -110,4 +109,3 @@ public: }; } -} diff --git a/searchlib/src/vespa/searchlib/datastore/array_store.hpp b/searchlib/src/vespa/searchlib/datastore/array_store.hpp index 33156c61fa5..8478c22903f 100644 --- a/searchlib/src/vespa/searchlib/datastore/array_store.hpp +++ b/searchlib/src/vespa/searchlib/datastore/array_store.hpp @@ -6,8 +6,7 @@ #include "datastore.hpp" #include -namespace search { -namespace datastore { +namespace search::datastore { constexpr size_t MIN_BUFFER_CLUSTERS = 8192; @@ -199,4 +198,4 @@ ArrayStore::optimizedConfigForHugePage(size_t maxSmallArraySize, } } -} + diff --git a/searchlib/src/vespa/searchlib/datastore/array_store_config.h b/searchlib/src/vespa/searchlib/datastore/array_store_config.h index a638541f97e..849ca53f35c 100644 --- a/searchlib/src/vespa/searchlib/datastore/array_store_config.h +++ b/searchlib/src/vespa/searchlib/datastore/array_store_config.h @@ -5,8 +5,7 @@ #include #include -namespace search { -namespace datastore { +namespace search::datastore { /** * Config specifying layout and buffer allocation strategy for an array store. @@ -66,4 +65,3 @@ public: }; } -} diff --git a/searchlib/src/vespa/searchlib/datastore/buffer_type.cpp b/searchlib/src/vespa/searchlib/datastore/buffer_type.cpp index 6023171abb5..5eca3719002 100644 --- a/searchlib/src/vespa/searchlib/datastore/buffer_type.cpp +++ b/searchlib/src/vespa/searchlib/datastore/buffer_type.cpp @@ -3,8 +3,7 @@ #include "buffer_type.h" #include -namespace search { -namespace datastore { +namespace search::datastore { BufferTypeBase::BufferTypeBase(uint32_t clusterSize, uint32_t minClusters, @@ -123,6 +122,5 @@ BufferTypeBase::calcClustersToAlloc(uint32_t bufferId, size_t sizeNeeded, bool r return wantClusters; } -} // namespace datastore -} // namespace search +} diff --git a/searchlib/src/vespa/searchlib/datastore/buffer_type.h b/searchlib/src/vespa/searchlib/datastore/buffer_type.h index e41e45f54c0..92dcbcc9e8d 100644 --- a/searchlib/src/vespa/searchlib/datastore/buffer_type.h +++ b/searchlib/src/vespa/searchlib/datastore/buffer_type.h @@ -6,8 +6,7 @@ #include #include -namespace search { -namespace datastore { +namespace search::datastore { /** * Class used manage allocation and de-allocation of a specific data type in @@ -163,6 +162,4 @@ BufferType::cleanHold(void *buffer, uint64_t offset, uint64_t len, Cl } } -} // namespace search::datastore -} // namespace search - +} diff --git a/searchlib/src/vespa/searchlib/datastore/bufferstate.cpp b/searchlib/src/vespa/searchlib/datastore/bufferstate.cpp index 09c7cf65bb6..cf6c59b69c4 100644 --- a/searchlib/src/vespa/searchlib/datastore/bufferstate.cpp +++ b/searchlib/src/vespa/searchlib/datastore/bufferstate.cpp @@ -5,8 +5,7 @@ using vespalib::alloc::Alloc; -namespace search { -namespace datastore { +namespace search::datastore { BufferState::FreeListList::~FreeListList() { @@ -251,6 +250,5 @@ BufferState::fallbackResize(uint32_t bufferId, std::atomic_thread_fence(std::memory_order_release); } -} // namespace datastore -} // namespace search +} diff --git a/searchlib/src/vespa/searchlib/datastore/bufferstate.h b/searchlib/src/vespa/searchlib/datastore/bufferstate.h index 5ff17d3bc81..904a5bbb16a 100644 --- a/searchlib/src/vespa/searchlib/datastore/bufferstate.h +++ b/searchlib/src/vespa/searchlib/datastore/bufferstate.h @@ -8,8 +8,7 @@ #include #include -namespace search { -namespace datastore { +namespace search::datastore { /** * Represents a memory allocated buffer (used in a data store) with its state. @@ -182,7 +181,4 @@ public: }; - -} // namespace search::datastore -} // namespace search - +} diff --git a/searchlib/src/vespa/searchlib/datastore/datastore.cpp b/searchlib/src/vespa/searchlib/datastore/datastore.cpp index 2f4e24c5d1a..5def7461e90 100644 --- a/searchlib/src/vespa/searchlib/datastore/datastore.cpp +++ b/searchlib/src/vespa/searchlib/datastore/datastore.cpp @@ -5,13 +5,11 @@ #include #include -namespace search { -namespace datastore { +namespace search::datastore { template class DataStoreT >; -} // namespace datastore -} // namespace search +} template void vespalib::Array::increase(size_t); template class search::attribute::RcuVector; diff --git a/searchlib/src/vespa/searchlib/datastore/datastore.h b/searchlib/src/vespa/searchlib/datastore/datastore.h index 0e3518960ed..26826ea0408 100644 --- a/searchlib/src/vespa/searchlib/datastore/datastore.h +++ b/searchlib/src/vespa/searchlib/datastore/datastore.h @@ -7,8 +7,7 @@ #include "free_list_allocator.h" #include "raw_allocator.h" -namespace search { -namespace btree { +namespace search::btree { template struct DefaultReclaimer { @@ -17,9 +16,9 @@ struct DefaultReclaimer { } }; -} // namespace btree +} -namespace datastore { +namespace search::datastore { template > class DataStoreT : public DataStoreBase @@ -108,6 +107,4 @@ public: extern template class DataStoreT >; -} // namespace search::datastore -} // namespace search - +} diff --git a/searchlib/src/vespa/searchlib/datastore/datastore.hpp b/searchlib/src/vespa/searchlib/datastore/datastore.hpp index f0fc451d952..de9bd1bc161 100644 --- a/searchlib/src/vespa/searchlib/datastore/datastore.hpp +++ b/searchlib/src/vespa/searchlib/datastore/datastore.hpp @@ -8,8 +8,7 @@ #include "raw_allocator.hpp" #include -namespace search { -namespace datastore { +namespace search::datastore { template DataStoreT::DataStoreT() @@ -193,6 +192,5 @@ DataStore::getEntry(EntryRef ref) const extern template class DataStoreT >; -} // namespace search::datastore -} // namespace search +} diff --git a/searchlib/src/vespa/searchlib/datastore/datastorebase.cpp b/searchlib/src/vespa/searchlib/datastore/datastorebase.cpp index 5cc1fecf798..d344c79d50e 100644 --- a/searchlib/src/vespa/searchlib/datastore/datastorebase.cpp +++ b/searchlib/src/vespa/searchlib/datastore/datastorebase.cpp @@ -5,8 +5,7 @@ using vespalib::GenerationHeldBase; -namespace search { -namespace datastore { +namespace search::datastore { namespace { @@ -517,5 +516,5 @@ DataStoreBase::startCompactWorstBuffers(bool compactMemory, bool compactAddressS return result; } -} // namespace datastore -} // namespace search +} + diff --git a/searchlib/src/vespa/searchlib/datastore/datastorebase.h b/searchlib/src/vespa/searchlib/datastore/datastorebase.h index 5add42da919..75e1ec8fb85 100644 --- a/searchlib/src/vespa/searchlib/datastore/datastorebase.h +++ b/searchlib/src/vespa/searchlib/datastore/datastorebase.h @@ -9,8 +9,7 @@ #include #include -namespace search { -namespace datastore { +namespace search::datastore { class DataStoreBase { @@ -345,6 +344,4 @@ public: }; -} // namespace datastore -} // namespace search - +} diff --git a/searchlib/src/vespa/searchlib/datastore/entryref.h b/searchlib/src/vespa/searchlib/datastore/entryref.h index 865dc3ed287..c8d7ffe8b66 100644 --- a/searchlib/src/vespa/searchlib/datastore/entryref.h +++ b/searchlib/src/vespa/searchlib/datastore/entryref.h @@ -2,10 +2,9 @@ #pragma once -#include +#include -namespace search { -namespace datastore { +namespace search::datastore { class EntryRef { protected: @@ -59,6 +58,4 @@ public: static uint64_t pad(uint64_t val) { return (-val & PadConstant); } }; -} // namespace search::datastore -} // namespace search - +} diff --git a/searchlib/src/vespa/searchlib/datastore/free_list_allocator.h b/searchlib/src/vespa/searchlib/datastore/free_list_allocator.h index 92dd02f9103..a23cb71b90c 100644 --- a/searchlib/src/vespa/searchlib/datastore/free_list_allocator.h +++ b/searchlib/src/vespa/searchlib/datastore/free_list_allocator.h @@ -4,8 +4,7 @@ #include "allocator.h" -namespace search { -namespace datastore { +namespace search::datastore { /** * Allocator used to allocate entries of a specific type in an underlying data store @@ -34,4 +33,3 @@ public: }; } -} diff --git a/searchlib/src/vespa/searchlib/datastore/free_list_allocator.hpp b/searchlib/src/vespa/searchlib/datastore/free_list_allocator.hpp index 6ed1c324b06..5db3e49119f 100644 --- a/searchlib/src/vespa/searchlib/datastore/free_list_allocator.hpp +++ b/searchlib/src/vespa/searchlib/datastore/free_list_allocator.hpp @@ -5,8 +5,7 @@ #include "free_list_allocator.h" #include "bufferstate.h" -namespace search { -namespace datastore { +namespace search::datastore { template FreeListAllocator::FreeListAllocator(DataStoreBase &store, uint32_t typeId) @@ -102,4 +101,4 @@ FreeListAllocator::allocArray(size_t size) } } -} + diff --git a/searchlib/src/vespa/searchlib/datastore/handle.h b/searchlib/src/vespa/searchlib/datastore/handle.h index b6119f4446d..c0dce8d3d75 100644 --- a/searchlib/src/vespa/searchlib/datastore/handle.h +++ b/searchlib/src/vespa/searchlib/datastore/handle.h @@ -4,8 +4,7 @@ #include "entryref.h" -namespace search { -namespace datastore { +namespace search::datastore { /** * Handle to data allocated in a data store and a EntryRef used for read-only access to data later. @@ -20,4 +19,3 @@ struct Handle }; } -} diff --git a/searchlib/src/vespa/searchlib/datastore/i_compaction_context.h b/searchlib/src/vespa/searchlib/datastore/i_compaction_context.h index afa7d0dff47..aa537968f1c 100644 --- a/searchlib/src/vespa/searchlib/datastore/i_compaction_context.h +++ b/searchlib/src/vespa/searchlib/datastore/i_compaction_context.h @@ -4,8 +4,7 @@ #include -namespace search { -namespace datastore { +namespace search::datastore { /** * A compaction context is used when performing a compaction of data buffers in a data store. @@ -20,4 +19,3 @@ struct ICompactionContext { }; } -} diff --git a/searchlib/src/vespa/searchlib/datastore/raw_allocator.h b/searchlib/src/vespa/searchlib/datastore/raw_allocator.h index 31c2d959648..d0b7d1d1ca2 100644 --- a/searchlib/src/vespa/searchlib/datastore/raw_allocator.h +++ b/searchlib/src/vespa/searchlib/datastore/raw_allocator.h @@ -6,8 +6,7 @@ #include "entryref.h" #include "handle.h" -namespace search { -namespace datastore { +namespace search::datastore { /** * Allocator used to allocate raw buffers (EntryT *) in an underlying data store @@ -33,4 +32,3 @@ public: }; } -} diff --git a/searchlib/src/vespa/searchlib/datastore/raw_allocator.hpp b/searchlib/src/vespa/searchlib/datastore/raw_allocator.hpp index 4e43a212fbd..1c72d793ec6 100644 --- a/searchlib/src/vespa/searchlib/datastore/raw_allocator.hpp +++ b/searchlib/src/vespa/searchlib/datastore/raw_allocator.hpp @@ -5,8 +5,7 @@ #include "raw_allocator.h" #include "bufferstate.h" -namespace search { -namespace datastore { +namespace search::datastore { template RawAllocator::RawAllocator(DataStoreBase &store, uint32_t typeId) @@ -30,4 +29,4 @@ RawAllocator::alloc(size_t numElems, size_t extraElems) } } -} + diff --git a/searchlib/src/vespa/searchlib/datastore/unique_store.h b/searchlib/src/vespa/searchlib/datastore/unique_store.h index cd5f9c9e489..15dc485b1da 100644 --- a/searchlib/src/vespa/searchlib/datastore/unique_store.h +++ b/searchlib/src/vespa/searchlib/datastore/unique_store.h @@ -10,8 +10,7 @@ #include #include -namespace search { -namespace datastore { +namespace search::datastore { template class UniqueStoreBuilder; @@ -117,4 +116,3 @@ public: }; } -} diff --git a/searchlib/src/vespa/searchlib/datastore/unique_store.hpp b/searchlib/src/vespa/searchlib/datastore/unique_store.hpp index 37835f1ea3f..603b6e53a85 100644 --- a/searchlib/src/vespa/searchlib/datastore/unique_store.hpp +++ b/searchlib/src/vespa/searchlib/datastore/unique_store.hpp @@ -15,8 +15,7 @@ #include "unique_store_saver.hpp" #include -namespace search { -namespace datastore { +namespace search::datastore { constexpr size_t NUMCLUSTERS_FOR_NEW_UNIQUESTORE_BUFFER = 1024u; @@ -252,4 +251,3 @@ UniqueStore::getNumUniques() const } } -} diff --git a/searchlib/src/vespa/searchlib/datastore/unique_store_builder.h b/searchlib/src/vespa/searchlib/datastore/unique_store_builder.h index f6bac2eb869..0a3ec447e67 100644 --- a/searchlib/src/vespa/searchlib/datastore/unique_store_builder.h +++ b/searchlib/src/vespa/searchlib/datastore/unique_store_builder.h @@ -4,8 +4,7 @@ #include "unique_store.h" -namespace search { -namespace datastore { +namespace search::datastore { /** * Builder for related UniqueStore class. @@ -45,4 +44,3 @@ public: }; } -} diff --git a/searchlib/src/vespa/searchlib/datastore/unique_store_builder.hpp b/searchlib/src/vespa/searchlib/datastore/unique_store_builder.hpp index 5182942f301..9445eb3a9f2 100644 --- a/searchlib/src/vespa/searchlib/datastore/unique_store_builder.hpp +++ b/searchlib/src/vespa/searchlib/datastore/unique_store_builder.hpp @@ -11,8 +11,7 @@ #include #include -namespace search { -namespace datastore { +namespace search::datastore { template UniqueStoreBuilder::UniqueStoreBuilder(DataStoreType &store, uint32_t typeId, Dictionary &dict, uint32_t uniqueValuesHint) @@ -57,4 +56,4 @@ UniqueStoreBuilder::makeDictionary() } } -} + diff --git a/searchlib/src/vespa/searchlib/datastore/unique_store_saver.h b/searchlib/src/vespa/searchlib/datastore/unique_store_saver.h index 74719fe83fa..6fdcf2da83a 100644 --- a/searchlib/src/vespa/searchlib/datastore/unique_store_saver.h +++ b/searchlib/src/vespa/searchlib/datastore/unique_store_saver.h @@ -4,8 +4,7 @@ #include "unique_store.h" -namespace search { -namespace datastore { +namespace search::datastore { /** * Saver for related UniqueStore class. @@ -50,4 +49,3 @@ public: }; } -} diff --git a/searchlib/src/vespa/searchlib/datastore/unique_store_saver.hpp b/searchlib/src/vespa/searchlib/datastore/unique_store_saver.hpp index 0b54d454c14..3377b674930 100644 --- a/searchlib/src/vespa/searchlib/datastore/unique_store_saver.hpp +++ b/searchlib/src/vespa/searchlib/datastore/unique_store_saver.hpp @@ -4,8 +4,7 @@ #include "unique_store_saver.h" -namespace search { -namespace datastore { +namespace search::datastore { template UniqueStoreSaver::UniqueStoreSaver(const Dictionary &dict, const DataStoreBase &store) @@ -46,4 +45,3 @@ UniqueStoreSaver::enumerateValues() } } -} -- cgit v1.2.3