Commit eb9847c8 authored by Alan Frindell's avatar Alan Frindell Committed by Dave Watson

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
parent b01aec06
...@@ -493,7 +493,6 @@ int Subprocess::runChild(const char* executable, ...@@ -493,7 +493,6 @@ int Subprocess::runChild(const char* executable,
char** argv, char** env, char** argv, char** env,
const Options& options) const { const Options& options) const {
// Now, finally, exec. // Now, finally, exec.
int r;
if (options.usePath_) { if (options.usePath_) {
::execvp(executable, argv); ::execvp(executable, argv);
} else { } else {
...@@ -818,4 +817,3 @@ Initializer initializer; ...@@ -818,4 +817,3 @@ Initializer initializer;
} // namespace } // namespace
} // namespace folly } // namespace folly
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment