From: Alan Frindell <afrind@fb.com>
Date: Mon, 29 Sep 2014 20:01:49 +0000 (-0700)
Subject: Fix compiler warning in Subprocess
X-Git-Tag: v0.22.0~312
X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=eb9847c86cc7de3574a480d071f773d140c34e49;p=folly.git

Fix compiler warning in Subprocess

Summary: I'm going to have hhvm depend on Subprocess, found this build error in open source.  I figured the intent was to set r to the return code.  I could also delete the whole thing since it's unused.

Test Plan: Unit tests

Reviewed By: tudorb@fb.com

Subscribers: trunkagent, mwilliams, doug, njormrod

FB internal diff: D1583626
---

diff --git a/folly/Subprocess.cpp b/folly/Subprocess.cpp
index 4cd2c7c1..c17e8849 100644
--- a/folly/Subprocess.cpp
+++ b/folly/Subprocess.cpp
@@ -493,7 +493,6 @@ int Subprocess::runChild(const char* executable,
                          char** argv, char** env,
                          const Options& options) const {
   // Now, finally, exec.
-  int r;
   if (options.usePath_) {
     ::execvp(executable, argv);
   } else {
@@ -818,4 +817,3 @@ Initializer initializer;
 }  // namespace
 
 }  // namespace folly
-