CodesHandle::getBytes: use length instead of size to determine array size

Author: pgeier

src/metkit/codes/GRIBDecoder.cc

@@ -60,21 +60,14 @@ void GRIBDecoder::getMetadata(const eckit::message::Message& msg, eckit::message
                 break;
             }
             case eckit::message::ValueRepresentation::Native: {
-                // https://jira.ecmwf.int/browse/ECC-2166
-                if (name == "uuidOfHGrid") {
-                    // uuidOfHGrid returns size 1 although it contains 16 bytes
-                    gather.setValue(name, k.getString());
-                }
-                else {
-                    std::visit(
-                        [&](auto&& v) {
-                            using Type = std::decay_t<decltype(v)>;
-                            if constexpr (std::is_same_v<Type, std::string> || std::is_arithmetic_v<Type>) {
-                                gather.setValue(name, std::forward<decltype(v)>(v));
-                            }
-                        },
-                        k.get());
-                }
+                std::visit(
+                    [&](auto&& v) {
+                        using Type = std::decay_t<decltype(v)>;
+                        if constexpr (std::is_same_v<Type, std::string> || std::is_arithmetic_v<Type>) {
+                            gather.setValue(name, std::forward<decltype(v)>(v));
+                        }
+                    },
+                    k.get());
                 break;
             }
         }

src/metkit/codes/api/CodesAPI.cc

@@ -85,6 +85,7 @@ class OwningCodesHandle : public CodesHandle {
     void forceSet(const std::string& key, Span<const double> value) override;
     void forceSet(const std::string& key, Span<const float> value) override;
     size_t size(const std::string& key) const override;
+    size_t length(const std::string& key) const override;
     CodesValue get(const std::string& key) const override;
     NativeType type(const std::string& key) const override;
     long getLong(const std::string& key) const override;
@@ -214,6 +215,12 @@ size_t OwningCodesHandle::size(const std::string& key) const {
     return size;
 }
 
+size_t OwningCodesHandle::length(const std::string& key) const {
+    size_t length;
+    throwOnError(codes_get_length(raw(), key.c_str(), &length), Here(), "CodesHandle::length(string)", key);
+    return length;
+}
+
 /// Get the value of the key
 CodesValue OwningCodesHandle::get(const std::string& key) const {
     NativeType ktype = type(key);
@@ -345,7 +352,7 @@ std::vector<std::string> OwningCodesHandle::getStringArray(const std::string& ke
 
 std::vector<uint8_t> OwningCodesHandle::getBytes(const std::string& key) const {
     std::vector<uint8_t> ret;
-    std::size_t ksize = size(key);
+    std::size_t ksize = length(key);
     ret.resize(ksize);
     throwOnError(codes_get_bytes(raw(), key.c_str(), ret.data(), &ksize), Here(), "CodesHandle::getBytes(string)", key);
     ret.resize(ksize);

src/metkit/codes/api/CodesAPI.h

@@ -203,6 +203,14 @@ class CodesHandle {
     /// @return For given scalars 1 is returned. For given arrays the size of the array is returned..
     virtual size_t size(const std::string& key) const = 0;
 
+    /// Returns the number of chars/bytes contained for a given key
+    //  of type string/byte array.
+    ///
+    /// Can be used to determine if a field is storing a scalar or an array.
+    /// @param key Name of the field that is supposed to be inspected.
+    /// @return Number of chars/bytes of contained string/byte array
+    virtual size_t length(const std::string& key) const = 0;
+
     /// Get the value of the key.
     ///
     /// High-level functionality: