Fix Build: folly with -Wmissing-braces

Summary:
[Folly] Fix Build: `folly` with `-Wmissing-braces`.

Construction of `std::array` with list-initialization for the contained C array requires double braces, per the standard.

Compilers accept it with single braces, but will reject it when `-Wmissing-braces` is passed.

Reviewed By: igorsugak, meyering

Differential Revision: D4202629

fbshipit-source-id: e5b87a655e7f25e6cddb931dda28b172c768f227

diff --git a/folly/Array.h b/folly/Array.h
index 77faafe34d75e8c12752e5136fec2870efcd4f0a..7ca3b6ca49f169fda66baf2787a09e52a70deb08 100644 (file)
--- a/folly/Array.h
+++ b/folly/Array.h
@@ -53,7 +53,7 @@ template <typename D = void, typename... TList>
 constexpr array_detail::return_type<D, TList...> make_array(TList&&... t) {
   using value_type =
       typename array_detail::return_type_helper<D, TList...>::type;
-  return {static_cast<value_type>(std::forward<TList>(t))...};
+  return {{static_cast<value_type>(std::forward<TList>(t))...}};
 }
 
 } // !folly

diff --git a/folly/futures/test/FutureTest.cpp b/folly/futures/test/FutureTest.cpp
index 8a66184af8e5dceec1417c8eac57244ab9104f31..7d0275d6013d9f86eb7d8fd68e917cfb9b9963a3 100644 (file)
--- a/folly/futures/test/FutureTest.cpp
+++ b/folly/futures/test/FutureTest.cpp
@@ -620,7 +620,7 @@ TEST(Future, finishBigLambda) {
   // bulk_data, to be captured in the lambda passed to Future::then.
   // This is meant to force that the lambda can't be stored inside
   // the Future object.
-  std::array<char, sizeof(detail::Core<int>)> bulk_data = {0};
+  std::array<char, sizeof(detail::Core<int>)> bulk_data = {{0}};
 
   // suppress gcc warning about bulk_data not being used
   EXPECT_EQ(bulk_data[0], 0);

diff --git a/folly/test/RangeTest.cpp b/folly/test/RangeTest.cpp
index 5861b60f4ed29d31a5db0759affab4bc532abe75..827e085cd9645d06a8b98f34f6a851d1dafc3bff 100644 (file)
--- a/folly/test/RangeTest.cpp
+++ b/folly/test/RangeTest.cpp
@@ -1111,7 +1111,7 @@ TEST(RangeFunc, ConstexprCArray) {
 }
 
 TEST(RangeFunc, ConstexprStdArray) {
-  static constexpr const std::array<int, 4> numArray = {3, 17, 1, 9};
+  static constexpr const std::array<int, 4> numArray = {{3, 17, 1, 9}};
   constexpr const auto numArrayRange = range(numArray);
   EXPECT_EQ(17, numArrayRange[1]);
   constexpr const auto numArrayRangeSize = numArrayRange.size();