From: Marcus Holland-Moritz <mhx@fb.com>
Date: Thu, 7 Jul 2016 08:27:05 +0000 (-0700)
Subject: Handle conversion from float in toDynamic()
X-Git-Tag: 2016.07.26~67
X-Git-Url: http://plrg.eecs.uci.edu/git/?a=commitdiff_plain;h=e88735c87c71aaffd33d559af6aa1a1247dd253d;p=folly.git

Handle conversion from float in toDynamic()

Summary:
This adds the necessary ConversionHelper to enable float-to-double
conversion when using `toDynamic` on any type that contains `float`.

Fixes the added test case, which previously failed to compile.

Reviewed By: yfeldblum

Differential Revision: D3525942

fbshipit-source-id: d904dde5585316ea9a15e21430e91ac4e33116b9
---

diff --git a/folly/dynamic-inl.h b/folly/dynamic-inl.h
index 920f627c..08374f7b 100644
--- a/folly/dynamic-inl.h
+++ b/folly/dynamic-inl.h
@@ -107,14 +107,17 @@ namespace detail {
   > {
     typedef int64_t type;
   };
-  template<class T>
+  template <>
+  struct ConversionHelper<float> {
+    typedef double type;
+  };
+  template <class T>
   struct ConversionHelper<
-    T,
-    typename std::enable_if<
-      (!std::is_integral<T>::value || std::is_same<T,bool>::value) &&
-      !std::is_same<T,std::nullptr_t>::value
-    >::type
-  > {
+      T,
+      typename std::enable_if<
+          (!std::is_integral<T>::value || std::is_same<T, bool>::value) &&
+          !std::is_same<T, float>::value &&
+          !std::is_same<T, std::nullptr_t>::value>::type> {
     typedef T type;
   };
   template<class T>
diff --git a/folly/test/DynamicConverterTest.cpp b/folly/test/DynamicConverterTest.cpp
index 039ce198..1d45e67d 100644
--- a/folly/test/DynamicConverterTest.cpp
+++ b/folly/test/DynamicConverterTest.cpp
@@ -343,6 +343,12 @@ TEST(DynamicConverter, construct) {
     EXPECT_EQ(d, toDynamic(c));
   }
 
+  {
+    vector<float> c{1.0f, 2.0f, 4.0f};
+    dynamic d = dynamic::array(1.0, 2.0, 4.0);
+    EXPECT_EQ(d, toDynamic(c));
+  }
+
   {
     map<int, int> c { { 2, 4 }, { 3, 9 } };
     dynamic d = dynamic::object(2, 4)(3, 9);