From: Nathan Bronson <ngbronson@fb.com>
Date: Thu, 10 Aug 2017 19:35:20 +0000 (-0700)
Subject: small_vector default constructor shouldn't be explicit
X-Git-Tag: v2017.08.14.00^0
X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=3b856e5082621b7b9479736a44a08254d0c77fb6;p=folly.git

small_vector default constructor shouldn't be explicit

Summary:
Containers should not tag their default constructor explicit,
because it means that you can't return an empty one with "return {};".
This diff removes the explicit tag from small_vector's default
constructor.

Reviewed By: meyering

Differential Revision: D5602749

fbshipit-source-id: 2a4a356134c69d01e3b040719dc6ffca0fe97e75
---

diff --git a/folly/small_vector.h b/folly/small_vector.h
index f28f4dfb..64add098 100644
--- a/folly/small_vector.h
+++ b/folly/small_vector.h
@@ -401,7 +401,7 @@ class small_vector
   typedef std::reverse_iterator<iterator>       reverse_iterator;
   typedef std::reverse_iterator<const_iterator> const_reverse_iterator;
 
-  explicit small_vector() = default;
+  small_vector() = default;
 
   small_vector(small_vector const& o) {
     auto n = o.size();