folly/wangle/channel/ChannelPipeline.h: avoid -Wsign-compare errors (trivial)
authorJim Meyering <meyering@fb.com>
Tue, 6 Jan 2015 23:15:46 +0000 (15:15 -0800)
committerViswanath Sivakumar <viswanath@fb.com>
Tue, 13 Jan 2015 19:01:04 +0000 (11:01 -0800)
Summary:
* folly/wangle/channel/ChannelPipeline.h: Declare for-loop indices
to be of type size_t(not int), to match size of upper bound.
Otherwise, gcc-4.9 fails with e.g.,
folly/wangle/channel/ChannelPipeline.h:126:23: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare]

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: davejwatson@fb.com

Subscribers: fugalh, folly-diffs@

FB internal diff: D1767783

Tasks: 5941250

Signature: t1:1767783:1420587925:d662b75e6a62ebd5bacdde28ad6e1da22ef777ac

folly/wangle/channel/ChannelPipeline.h

index 07d10ba8838e4e05ff4fcb2e9a97b0afd709fe61..6536efacd41149b63fce13b473c47c2fccc8bccb 100644 (file)
@@ -123,7 +123,7 @@ class ChannelPipeline<R, W> : public DelayedDestruction {
       return;
     }
 
-    for (int i = 0; i < ctxs_.size() - 1; i++) {
+    for (size_t i = 0; i < ctxs_.size() - 1; i++) {
       ctxs_[i]->link(ctxs_[i+1].get());
     }
 
@@ -293,7 +293,7 @@ class ChannelPipeline<R, W, Handler, Handlers...>
     }
 
     if (!ctxs_.empty()) {
-      for (int i = 0; i < ctxs_.size() - 1; i++) {
+      for (size_t i = 0; i < ctxs_.size() - 1; i++) {
         ctxs_[i]->link(ctxs_[i+1].get());
       }
       ctxs_.back()->link(&ctx_);