folly/wangle/futures/Future-inl.h: avoid -Wsign-compare errors

Summary:
* folly/wangle/futures/Future-inl.h (whenAll): Do not compare
signed (std::distance) with unsigned(size_t): add an assertion
to eliminate the possibility of negative, then assign to a new
size_t "n".

Test Plan:
Run this and note there are fewer errors than before:
fbconfig --platform-all=gcc-4.9-glibc-2.20 -r folly && fbmake dbgo

Reviewed By: jsedgwick@fb.com

Subscribers: trunkagent, fugalh, folly-diffs@

FB internal diff: D1770668

Tasks: 5941250

Signature: t1:1770668:1420736965:fc722657a2ec523052fd96c33d65d5ca24aae61c

diff --git a/folly/wangle/futures/Future-inl.h b/folly/wangle/futures/Future-inl.h
index 66cadbcf7f02f1c70e1504e4f056278d81b39ee0..61c386ad80c8cbcd52f9b95c315470d89e244b11 100644 (file)
--- a/folly/wangle/futures/Future-inl.h
+++ b/folly/wangle/futures/Future-inl.h
@@ -546,10 +546,10 @@ whenAll(InputIterator first, InputIterator last)
   typedef
     typename std::iterator_traits<InputIterator>::value_type::value_type T;
 
-  auto n = std::distance(first, last);
-  if (n == 0) {
+  if (first >= last) {
     return makeFuture(std::vector<Try<T>>());
   }
+  size_t n = std::distance(first, last);
 
   auto ctx = new detail::WhenAllContext<T>();