Skip to content

Commit

Permalink
fixing contbuild_memcache_build failure
Browse files Browse the repository at this point in the history
Summary: Fix to mcrouter/lib/carbon/connection/PooledCarbonConnectionImpl.h:47:9: error: operation on ‘clientId’ may be undefined [-Werror=sequence-point]

Reviewed By: jmswen

Differential Revision: D4433604

fbshipit-source-id: d7aee84c591da42aec7ea51c6aad252014500c55
  • Loading branch information
glamtechie authored and facebook-github-bot committed Jan 19, 2017
1 parent b40c6c9 commit 6d49b6b
Showing 1 changed file with 1 addition and 1 deletion.
Expand Up @@ -44,7 +44,7 @@ class PooledCarbonConnectionImpl {
auto clientId = folly::Random::rand32(connections_.size());
for (const Request& req : reqs) {
connections_[clientId]->sendRequestOne(req, cb);
clientId = (++clientId) % (connections_.size());
clientId = (clientId + 1) % connections_.size();
}
} else {
connections_[folly::Random::rand32(connections_.size())]
Expand Down

0 comments on commit 6d49b6b

Please sign in to comment.