CXX-2625 Address sign conversion warnings for literal argument to make_unique<T[]>() (#1044)

Author: eramongodb

src/bsoncxx/include/bsoncxx/v_noabi/bsoncxx/stdx/make_unique.hpp

@@ -102,11 +102,25 @@ struct make_unique_impl<T&&> {};
 template <typename T,
           typename... Args,
           typename Impl = detail::make_unique_impl<T>,
-          typename = decltype(Impl::make(std::true_type{}, std::declval<Args>()...))>
+          typename std::enable_if<!std::is_array<T>::value,
+                                  decltype(Impl::make(std::true_type{}, std::declval<Args>()...),
+                                           void())>::type* = nullptr>
 std::unique_ptr<T> make_unique(Args&&... args) {
     return Impl::make(std::true_type{}, std::forward<Args>(args)...);
 }
 
+/**
+ * @copydoc bsoncxx::v_noabi::stdx::make_unique
+ */
+template <typename T,
+          typename Impl = detail::make_unique_impl<T>,
+          typename std::enable_if<std::is_array<T>::value,
+                                  decltype(Impl::make(std::true_type{}, std::declval<std::size_t>()),
+                                           void())>::type* = nullptr>
+std::unique_ptr<T> make_unique(std::size_t count) {
+    return Impl::make(std::true_type{}, count);
+}
+
 /**
  * @brief Create a new std::unique_ptr that points-to a default-initialized instance of the given
  * type.
@@ -125,11 +139,25 @@ std::unique_ptr<T> make_unique(Args&&... args) {
 template <typename T,
           typename... Args,
           typename Impl = detail::make_unique_impl<T>,
-          typename = decltype(Impl::make(std::false_type{}, std::declval<Args>()...))>
+          typename std::enable_if<!std::is_array<T>::value,
+                                  decltype(Impl::make(std::false_type{}, std::declval<Args>()...),
+                                           void())>::type* = nullptr>
 std::unique_ptr<T> make_unique_for_overwrite(Args&&... args) {
     return Impl::make(std::false_type{}, std::forward<Args>(args)...);
 }
 
+/**
+ * @copydoc bsoncxx::v_noabi::stdx::make_unique_for_overwrite
+ */
+template <typename T,
+          typename Impl = detail::make_unique_impl<T>,
+          typename std::enable_if<std::is_array<T>::value,
+                                  decltype(Impl::make(std::false_type{}, std::declval<std::size_t>()),
+                                           void())>::type* = nullptr>
+std::unique_ptr<T> make_unique_for_overwrite(std::size_t count) {
+    return Impl::make(std::false_type{}, count);
+}
+
 }  // namespace stdx
 BSONCXX_INLINE_NAMESPACE_END
 }  // namespace bsoncxx