SingleKeyValue: allow seamless construction from the underlying type

SingleKeyValue is a tiny wrapper and supposed to be discreet.
Having to explicitly (even if only with braces) construct its objects
stands in the way of readability on the consuming side of the code and
sometimes prevents direct initialisation of event objects without
constructors getting some kind of ContentParamTs parameter pack where
a single content_type argument would suffice otherwise.

1 files changed, 9 insertions, 0 deletions

diff --git a/lib/events/single_key_value.h b/lib/events/single_key_value.h
index 75ca8cd2..5edff3b1 100644
--- a/lib/events/single_key_value.h
+++ b/lib/events/single_key_value.h
@@ -7,6 +7,15 @@ namespace Quotient {
 namespace EventContent {
     template <typename T, const QLatin1String* KeyStr>
     struct SingleKeyValue {
+        // NOLINTBEGIN(google-explicit-constructor): that check should learn
+        //                                           about explicit(false)
+        QUO_IMPLICIT SingleKeyValue(const T& v = {})
+            : value { v }
+        {}
+        QUO_IMPLICIT SingleKeyValue(T&& v)
+            : value { std::move(v) }
+        {}
+        // NOLINTEND(google-explicit-constructor)
         T value;
     };
 } // namespace EventContent