Use auto, braced initialiser and noexcept

2 files changed, 3 insertions, 3 deletions

diff --git a/document/src/vespa/document/fieldvalue/boolfieldvalue.cpp b/document/src/vespa/document/fieldvalue/boolfieldvalue.cpp
index 90b4b1711f1..b0b77fe2bf6 100644
--- a/document/src/vespa/document/fieldvalue/boolfieldvalue.cpp
+++ b/document/src/vespa/document/fieldvalue/boolfieldvalue.cpp
@@ -29,7 +29,7 @@ int
 BoolFieldValue::compare(const FieldValue&rhs) const {
     int diff = FieldValue::compare(rhs);
     if (diff != 0) return diff;
-    const BoolFieldValue &o = static_cast<const BoolFieldValue &>(rhs);
+    const auto &o = static_cast<const BoolFieldValue &>(rhs);
     return (_value == o._value) ? 0 : _value ? 1 : -1;
 }
 
diff --git a/document/src/vespa/document/fieldvalue/fieldvalue.cpp b/document/src/vespa/document/fieldvalue/fieldvalue.cpp
index d1939f09a1d..1fef35fec0c 100644
--- a/document/src/vespa/document/fieldvalue/fieldvalue.cpp
+++ b/document/src/vespa/document/fieldvalue/fieldvalue.cpp
@@ -185,7 +185,7 @@ FieldValue::UP
 FieldValue::onGetNestedFieldValue(PathRange nested) const
 {
     (void) nested;
-    return FieldValue::UP();
+    return {};
 }
 
 ModificationStatus
@@ -221,7 +221,7 @@ namespace {
 class FieldValueFactory : public ComplexArrayT<FieldValue>::Factory
 {
 public:
-    FieldValueFactory(const DataType & dataType) : _dataType(&dataType) { }
+    explicit FieldValueFactory(const DataType & dataType) noexcept : _dataType(&dataType) { }
     FieldValue * create() override { return _dataType->createFieldValue().release(); }
     FieldValueFactory * clone() const override { return new FieldValueFactory(*this); }
 private: