Merge pull request #31055 from vespa-engine/balder/handle-tensor-not-present

Add final on MutableDenseValueView and use noexcept where applicable.

1 files changed, 7 insertions, 7 deletions

diff --git a/searchlib/src/vespa/searchlib/features/mutable_dense_value_view.h b/searchlib/src/vespa/searchlib/features/mutable_dense_value_view.h
index 4794de23b4a..52576841093 100644
--- a/searchlib/src/vespa/searchlib/features/mutable_dense_value_view.h
+++ b/searchlib/src/vespa/searchlib/features/mutable_dense_value_view.h
@@ -12,20 +12,20 @@ using namespace vespalib::eval;
 /**
  * A dense tensor with a cells reference that can be modified.
  */
-class MutableDenseValueView : public Value {
+class MutableDenseValueView final : public Value {
 private:
     const ValueType _type;
     TypedCells _cells;
 public:
-    MutableDenseValueView(const ValueType &type_in);
-    void setCells(TypedCells cells_in) {
+    explicit MutableDenseValueView(const ValueType &type_in);
+    void setCells(TypedCells cells_in) noexcept {
         assert(cells_in.type == _type.cell_type());
         _cells = cells_in;
     }
-    const ValueType &type() const final override { return _type; }
-    TypedCells cells() const final override { return _cells; }
-    const Index &index() const final override { return TrivialIndex::get(); }
-    vespalib::MemoryUsage get_memory_usage() const final override {
+    const ValueType &type() const override { return _type; }
+    TypedCells cells() const override { return _cells; }
+    const Index &index() const override { return TrivialIndex::get(); }
+    vespalib::MemoryUsage get_memory_usage() const override {
         return self_memory_usage<MutableDenseValueView>();
     }
 };