(Wangle) Remove unnecessary constructor argument to CollectAnyContext
authorHannes Roth <hannesr@fb.com>
Wed, 24 Jun 2015 17:20:39 +0000 (10:20 -0700)
committerSara Golemon <sgolemon@fb.com>
Wed, 24 Jun 2015 18:06:12 +0000 (11:06 -0700)
Summary: See title.

Reviewed By: @fugalh

Differential Revision: D2186833

folly/futures/Future-inl.h

index 7fdc09dae8dbaee41c66f8d18c24df1f146e09e2..9cacf19db2afd47b56583ea9e062875f2db27083 100644 (file)
@@ -660,12 +660,12 @@ collectAny(InputIterator first, InputIterator last) {
     typename std::iterator_traits<InputIterator>::value_type::value_type T;
 
   struct CollectAnyContext {
-    CollectAnyContext(size_t n) {};
+    CollectAnyContext() {};
     Promise<std::pair<size_t, Try<T>>> p;
     std::atomic<bool> done {false};
   };
 
-  auto ctx = std::make_shared<CollectAnyContext>(std::distance(first, last));
+  auto ctx = std::make_shared<CollectAnyContext>();
   mapSetCallback<T>(first, last, [ctx](size_t i, Try<T>&& t) {
     if (!ctx->done.exchange(true)) {
       ctx->p.setValue(std::make_pair(i, std::move(t)));