Fix compiler warning in Subprocess
authorAlan Frindell <afrind@fb.com>
Mon, 29 Sep 2014 20:01:49 +0000 (13:01 -0700)
committerDave Watson <davejwatson@fb.com>
Tue, 30 Sep 2014 23:16:25 +0000 (16:16 -0700)
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

folly/Subprocess.cpp

index 4cd2c7c1088b0021d2244477fd03134c2cf29e82..c17e8849e8f080a2a722b84a35b5317a6a68b2df 100644 (file)
@@ -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
-