diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2023-01-10 07:41:07 +0000 |
---|---|---|
committer | Henning Baldersheim <balder@yahoo-inc.com> | 2023-01-10 11:38:08 +0000 |
commit | 532395fd9a02c8ca77b1502a575eb710d374c91c (patch) | |
tree | 43aa15b3f512f47d69797821a7b29c47379cb764 /searchlib | |
parent | 288e97e863e7e6c0a52b0ed1ea2d74d69a3f30ae (diff) |
Use const to signal members that will not change
Diffstat (limited to 'searchlib')
5 files changed, 5 insertions, 6 deletions
diff --git a/searchlib/src/vespa/searchlib/memoryindex/field_index_base.h b/searchlib/src/vespa/searchlib/memoryindex/field_index_base.h index 93fe77ba50b..2d6d367af3b 100644 --- a/searchlib/src/vespa/searchlib/memoryindex/field_index_base.h +++ b/searchlib/src/vespa/searchlib/memoryindex/field_index_base.h @@ -78,7 +78,7 @@ protected: GenerationHandler _generationHandler; DictionaryTree _dict; FeatureStore _featureStore; - uint32_t _fieldId; + const uint32_t _fieldId; FieldIndexRemover _remover; std::unique_ptr<IOrderedFieldIndexInserter> _inserter; index::FieldLengthCalculator _calculator; diff --git a/searchlib/src/vespa/searchlib/memoryindex/field_index_collection.h b/searchlib/src/vespa/searchlib/memoryindex/field_index_collection.h index 7a65195e843..a9f597e6296 100644 --- a/searchlib/src/vespa/searchlib/memoryindex/field_index_collection.h +++ b/searchlib/src/vespa/searchlib/memoryindex/field_index_collection.h @@ -28,7 +28,7 @@ private: using GenerationHandler = vespalib::GenerationHandler; std::vector<std::unique_ptr<IFieldIndex>> _fieldIndexes; - uint32_t _numFields; + const uint32_t _numFields; public: FieldIndexCollection(const index::Schema& schema, const index::IFieldLengthInspector& inspector); diff --git a/searchlib/src/vespa/searchlib/memoryindex/field_inverter.h b/searchlib/src/vespa/searchlib/memoryindex/field_inverter.h index e68f664603e..7995dc56de8 100644 --- a/searchlib/src/vespa/searchlib/memoryindex/field_inverter.h +++ b/searchlib/src/vespa/searchlib/memoryindex/field_inverter.h @@ -97,7 +97,7 @@ private: class ElemInfo { public: - int32_t _weight; + const int32_t _weight; uint32_t _len; uint32_t _field_length; @@ -161,7 +161,7 @@ private: using UInt32Vector = std::vector<uint32_t, vespalib::allocator_large<uint32_t>>; // Current field state. - uint32_t _fieldId; // current field id + const uint32_t _fieldId; // current field id uint32_t _elem; // current element uint32_t _wpos; // current word pos uint32_t _docId; diff --git a/searchlib/src/vespa/searchlib/memoryindex/invert_task.h b/searchlib/src/vespa/searchlib/memoryindex/invert_task.h index b898deb5c16..a351fd2a10f 100644 --- a/searchlib/src/vespa/searchlib/memoryindex/invert_task.h +++ b/searchlib/src/vespa/searchlib/memoryindex/invert_task.h @@ -27,7 +27,7 @@ class InvertTask : public vespalib::Executor::Task const std::vector<std::unique_ptr<FieldInverter>>& _inverters; const std::vector<std::unique_ptr<UrlFieldInverter>>& _uri_inverters; const document::Document& _doc; - uint32_t _lid; + const uint32_t _lid; std::remove_reference_t<OnWriteDoneType> _on_write_done; public: InvertTask(const DocumentInverterContext& inv_context, const InvertContext& context, const std::vector<std::unique_ptr<FieldInverter>>& inverters, const std::vector<std::unique_ptr<UrlFieldInverter>>& uri_inverters, uint32_t lid, const document::Document& doc, OnWriteDoneType on_write_done); diff --git a/searchlib/src/vespa/searchlib/memoryindex/posting_list_entry.h b/searchlib/src/vespa/searchlib/memoryindex/posting_list_entry.h index ef0866e957b..a8cc7fce1f2 100644 --- a/searchlib/src/vespa/searchlib/memoryindex/posting_list_entry.h +++ b/searchlib/src/vespa/searchlib/memoryindex/posting_list_entry.h @@ -13,7 +13,6 @@ class InterleavedFeatures { protected: uint16_t _num_occs; uint16_t _field_length; - public: InterleavedFeatures() : _num_occs(0), |