Rename IKeyValue to KeyValueStorageBase.

Author: aentinger

src/util/storage/IKeyValue.hpp src/util/storage/KeyValueStorageBase.hppsrc/util/storage/IKeyValue.hpp renamed to src/util/storage/KeyValueStorageBase.hpp

@@ -30,15 +30,15 @@ enum class Error : std::uint8_t
 /// validate data integrity per key (e.g., using CRC and such).
 /// This interface is fully blocking and should only be used during initialization and shutdown,
 /// never during normal operation. Non-blocking adapters can be built on top of it.
-class IKeyValue
+class KeyValueStorageBase
 {
 public:
-    IKeyValue()                                    = default;
-    IKeyValue(const IKeyValue&)                    = delete;
-    IKeyValue(IKeyValue&&)                         = delete;
-    auto operator=(const IKeyValue&) -> IKeyValue& = delete;
-    auto operator=(IKeyValue&&) -> IKeyValue&      = delete;
-    virtual ~IKeyValue()                           = default;
+    KeyValueStorageBase()                                              = default;
+    KeyValueStorageBase(const KeyValueStorageBase&)                    = delete;
+    KeyValueStorageBase(KeyValueStorageBase&&)                         = delete;
+    auto operator=(const KeyValueStorageBase&) -> KeyValueStorageBase& = delete;
+    auto operator=(KeyValueStorageBase&&) -> KeyValueStorageBase&      = delete;
+    virtual ~KeyValueStorageBase()                                     = default;
 
     /// The return value is the number of bytes read into the buffer or the error.
     [[nodiscard]] virtual auto get(const std::string_view key, const std::size_t size, void* const data) const

src/util/storage/register_storage.hpp

@@ -9,7 +9,7 @@
 
 #if __GNUC__ >= 11
 
-#include "IKeyValue.hpp"
+#include "KeyValueStorageBase.hpp"
 
 #include <functional>
 
@@ -23,7 +23,7 @@ namespace cyphal::support
 /// Stored registers that are not present in the registry will not be loaded.
 /// The serialization format is simply the Cyphal DSDL.
 /// In case of error, only part of the registers may be loaded and the registry will be left in an inconsistent state.
-[[nodiscard]] inline std::optional<platform::storage::Error> load(const platform::storage::IKeyValue& kv,
+[[nodiscard]] inline std::optional<platform::storage::Error> load(const platform::storage::KeyValueStorageBase& kv,
                                                                   registry::IIntrospectableRegistry&  rgy)
 {
     for (std::size_t index = 0; index < rgy.size(); index++)
@@ -82,7 +82,7 @@ namespace cyphal::support
 /// The removal predicate, if provided, allows the caller to specify which registers need to be removed from the
 /// storage instead of being saved. This is useful for implementing the "factory reset" feature.
 template <typename ResetPredicate>
-[[nodiscard]] std::optional<platform::storage::Error> save(platform::storage::IKeyValue&            kv,
+[[nodiscard]] std::optional<platform::storage::Error> save(platform::storage::KeyValueStorageBase&            kv,
                                                            const registry::IIntrospectableRegistry& rgy,
                                                            ResetPredicate const reset_predicate)
 {
@@ -127,7 +127,7 @@ template <typename ResetPredicate>
     }
     return std::nullopt;
 }
-[[nodiscard]] inline std::optional<platform::storage::Error> save(platform::storage::IKeyValue&            kv,
+[[nodiscard]] inline std::optional<platform::storage::Error> save(platform::storage::KeyValueStorageBase&            kv,
                                                                   const registry::IIntrospectableRegistry& rgy)
 {
     return save(kv, rgy, [](std::string_view) { return false; });