summaryrefslogtreecommitdiffstats
path: root/Source/cmExecProgramCommand.cxx
diff options
context:
space:
mode:
authorKitware Robot <kwrobot@kitware.com>2016-05-16 14:34:04 (GMT)
committerBrad King <brad.king@kitware.com>2016-05-16 20:05:19 (GMT)
commitd9fd2f5402eeaa345691313658e02b51038f570b (patch)
treedca71b9a7e267f4c6300da3eb770415381726785 /Source/cmExecProgramCommand.cxx
parent82df6deaafb36cbbfd450202bb20b320f637751a (diff)
downloadCMake-d9fd2f5402eeaa345691313658e02b51038f570b.zip
CMake-d9fd2f5402eeaa345691313658e02b51038f570b.tar.gz
CMake-d9fd2f5402eeaa345691313658e02b51038f570b.tar.bz2
Revise C++ coding style using clang-format
Run the `Utilities/Scripts/clang-format.bash` script to update all our C++ code to a new style defined by `.clang-format`. Use `clang-format` version 3.8. * If you reached this commit for a line in `git blame`, re-run the blame operation starting at the parent of this commit to see older history for the content. * See the parent commit for instructions to rebase a change across this style transition commit.
Diffstat (limited to 'Source/cmExecProgramCommand.cxx')
-rw-r--r--Source/cmExecProgramCommand.cxx221
1 files changed, 83 insertions, 138 deletions
diff --git a/Source/cmExecProgramCommand.cxx b/Source/cmExecProgramCommand.cxx
index 1109737..f3f8c2f 100644
--- a/Source/cmExecProgramCommand.cxx
+++ b/Source/cmExecProgramCommand.cxx
@@ -16,14 +16,13 @@
#include <cmsys/Process.h>
// cmExecProgramCommand
-bool cmExecProgramCommand
-::InitialPass(std::vector<std::string> const& args, cmExecutionStatus &)
+bool cmExecProgramCommand::InitialPass(std::vector<std::string> const& args,
+ cmExecutionStatus&)
{
- if(args.size() < 1 )
- {
+ if (args.size() < 1) {
this->SetError("called with incorrect number of arguments");
return false;
- }
+ }
std::string arguments;
bool doingargs = false;
int count = 0;
@@ -31,131 +30,102 @@ bool cmExecProgramCommand
bool haveoutput_variable = false;
std::string return_variable;
bool havereturn_variable = false;
- for(size_t i=0; i < args.size(); ++i)
- {
- if(args[i] == "OUTPUT_VARIABLE")
- {
+ for (size_t i = 0; i < args.size(); ++i) {
+ if (args[i] == "OUTPUT_VARIABLE") {
count++;
doingargs = false;
havereturn_variable = false;
haveoutput_variable = true;
- }
- else if ( haveoutput_variable )
- {
- if (!output_variable.empty())
- {
+ } else if (haveoutput_variable) {
+ if (!output_variable.empty()) {
this->SetError("called with incorrect number of arguments");
return false;
- }
+ }
output_variable = args[i];
haveoutput_variable = false;
- count ++;
- }
- else if(args[i] == "RETURN_VALUE")
- {
+ count++;
+ } else if (args[i] == "RETURN_VALUE") {
count++;
doingargs = false;
haveoutput_variable = false;
havereturn_variable = true;
- }
- else if ( havereturn_variable )
- {
- if (!return_variable.empty())
- {
+ } else if (havereturn_variable) {
+ if (!return_variable.empty()) {
this->SetError("called with incorrect number of arguments");
return false;
- }
+ }
return_variable = args[i];
havereturn_variable = false;
- count ++;
- }
- else if(args[i] == "ARGS")
- {
+ count++;
+ } else if (args[i] == "ARGS") {
count++;
havereturn_variable = false;
haveoutput_variable = false;
doingargs = true;
- }
- else if(doingargs)
- {
+ } else if (doingargs) {
arguments += args[i];
arguments += " ";
count++;
- }
}
+ }
std::string command;
- if(!arguments.empty())
- {
+ if (!arguments.empty()) {
command = cmSystemTools::ConvertToRunCommandPath(args[0].c_str());
command += " ";
command += arguments;
- }
- else
- {
+ } else {
command = args[0];
- }
+ }
bool verbose = true;
- if(!output_variable.empty())
- {
+ if (!output_variable.empty()) {
verbose = false;
- }
+ }
int retVal = 0;
std::string output;
bool result = true;
- if(args.size() - count == 2)
- {
+ if (args.size() - count == 2) {
cmSystemTools::MakeDirectory(args[1].c_str());
result = cmExecProgramCommand::RunCommand(command.c_str(), output, retVal,
args[1].c_str(), verbose);
- }
- else
- {
- result = cmExecProgramCommand::RunCommand(command.c_str(), output,
- retVal, 0, verbose);
- }
- if(!result)
- {
+ } else {
+ result = cmExecProgramCommand::RunCommand(command.c_str(), output, retVal,
+ 0, verbose);
+ }
+ if (!result) {
retVal = -1;
- }
+ }
- if (!output_variable.empty())
- {
+ if (!output_variable.empty()) {
std::string::size_type first = output.find_first_not_of(" \n\t\r");
std::string::size_type last = output.find_last_not_of(" \n\t\r");
- if(first == std::string::npos)
- {
+ if (first == std::string::npos) {
first = 0;
- }
- if(last == std::string::npos)
- {
- last = output.size()-1;
- }
+ }
+ if (last == std::string::npos) {
+ last = output.size() - 1;
+ }
- std::string coutput = std::string(output, first, last-first+1);
+ std::string coutput = std::string(output, first, last - first + 1);
this->Makefile->AddDefinition(output_variable, coutput.c_str());
- }
+ }
- if (!return_variable.empty())
- {
+ if (!return_variable.empty()) {
char buffer[100];
sprintf(buffer, "%d", retVal);
this->Makefile->AddDefinition(return_variable, buffer);
- }
+ }
return true;
}
-bool cmExecProgramCommand::RunCommand(const char* command,
- std::string& output,
- int &retVal,
- const char* dir,
+bool cmExecProgramCommand::RunCommand(const char* command, std::string& output,
+ int& retVal, const char* dir,
bool verbose)
{
- if(cmSystemTools::GetRunCommandOutput())
- {
+ if (cmSystemTools::GetRunCommandOutput()) {
verbose = false;
- }
+ }
#if defined(_WIN32) && !defined(__CYGWIN__)
// if the command does not start with a quote, then
@@ -164,100 +134,82 @@ bool cmExecProgramCommand::RunCommand(const char* command,
// shell command like echo or dir
int count = 0;
std::string shortCmd;
- if(command[0] == '\"')
- {
+ if (command[0] == '\"') {
// count the number of quotes
- for(const char* s = command; *s != 0; ++s)
- {
- if(*s == '\"')
- {
+ for (const char* s = command; *s != 0; ++s) {
+ if (*s == '\"') {
count++;
- if(count > 2)
- {
+ if (count > 2) {
break;
- }
}
}
+ }
// if there are more than two double quotes use
// GetShortPathName, the cmd.exe program in windows which
// is used by system fails to execute if there are more than
// one set of quotes in the arguments
- if(count > 2)
- {
+ if (count > 2) {
cmsys::RegularExpression quoted("^\"([^\"]*)\"[ \t](.*)");
- if(quoted.find(command))
- {
+ if (quoted.find(command)) {
std::string cmd = quoted.match(1);
std::string args = quoted.match(2);
- if(! cmSystemTools::FileExists(cmd.c_str()) )
- {
+ if (!cmSystemTools::FileExists(cmd.c_str())) {
shortCmd = cmd;
- }
- else if(!cmSystemTools::GetShortPath(cmd.c_str(), shortCmd))
- {
- cmSystemTools::Error("GetShortPath failed for " , cmd.c_str());
+ } else if (!cmSystemTools::GetShortPath(cmd.c_str(), shortCmd)) {
+ cmSystemTools::Error("GetShortPath failed for ", cmd.c_str());
return false;
- }
+ }
shortCmd += " ";
shortCmd += args;
command = shortCmd.c_str();
- }
- else
- {
+ } else {
cmSystemTools::Error("Could not parse command line with quotes ",
command);
- }
}
}
+ }
#endif
// Allocate a process instance.
cmsysProcess* cp = cmsysProcess_New();
- if(!cp)
- {
+ if (!cp) {
cmSystemTools::Error("Error allocating process instance.");
return false;
- }
+ }
#if defined(_WIN32) && !defined(__CYGWIN__)
- if(dir)
- {
+ if (dir) {
cmsysProcess_SetWorkingDirectory(cp, dir);
- }
- if(cmSystemTools::GetRunCommandHideConsole())
- {
+ }
+ if (cmSystemTools::GetRunCommandHideConsole()) {
cmsysProcess_SetOption(cp, cmsysProcess_Option_HideWindow, 1);
- }
+ }
cmsysProcess_SetOption(cp, cmsysProcess_Option_Verbatim, 1);
- const char* cmd[] = {command, 0};
+ const char* cmd[] = { command, 0 };
cmsysProcess_SetCommand(cp, cmd);
#else
std::string commandInDir;
- if(dir)
- {
+ if (dir) {
commandInDir = "cd \"";
commandInDir += dir;
commandInDir += "\" && ";
commandInDir += command;
- }
- else
- {
+ } else {
commandInDir = command;
- }
+ }
#ifndef __VMS
commandInDir += " 2>&1";
#endif
command = commandInDir.c_str();
- if(verbose)
- {
+ if (verbose) {
cmSystemTools::Stdout("running ");
cmSystemTools::Stdout(command);
cmSystemTools::Stdout("\n");
- }
+ }
fflush(stdout);
fflush(stderr);
- const char* cmd[] = {"/bin/sh", "-c", command, 0};
+ const char* cmd[] = { "/bin/sh", "-c", command, 0 };
cmsysProcess_SetCommand(cp, cmd);
#endif
@@ -267,25 +219,21 @@ bool cmExecProgramCommand::RunCommand(const char* command,
int length;
char* data;
int p;
- while((p = cmsysProcess_WaitForData(cp, &data, &length, 0), p))
- {
- if(p == cmsysProcess_Pipe_STDOUT || p == cmsysProcess_Pipe_STDERR)
- {
- if(verbose)
- {
+ while ((p = cmsysProcess_WaitForData(cp, &data, &length, 0), p)) {
+ if (p == cmsysProcess_Pipe_STDOUT || p == cmsysProcess_Pipe_STDERR) {
+ if (verbose) {
cmSystemTools::Stdout(data, length);
- }
- output.append(data, length);
}
+ output.append(data, length);
}
+ }
// All output has been read. Wait for the process to exit.
cmsysProcess_WaitForExit(cp, 0);
// Check the result of running the process.
std::string msg;
- switch(cmsysProcess_GetState(cp))
- {
+ switch (cmsysProcess_GetState(cp)) {
case cmsysProcess_State_Exited:
retVal = cmsysProcess_GetExitValue(cp);
break;
@@ -303,29 +251,26 @@ bool cmExecProgramCommand::RunCommand(const char* command,
retVal = -1;
msg += "\nProcess terminated due to timeout.";
break;
- }
- if(!msg.empty())
- {
+ }
+ if (!msg.empty()) {
#if defined(_WIN32) && !defined(__CYGWIN__)
// Old Windows process execution printed this info.
msg += "\n\nfor command: ";
msg += command;
- if(dir)
- {
+ if (dir) {
msg += "\nin dir: ";
msg += dir;
- }
+ }
msg += "\n";
- if(verbose)
- {
+ if (verbose) {
cmSystemTools::Stdout(msg.c_str());
- }
+ }
output += msg;
#else
// Old UNIX process execution only put message in output.
output += msg;
#endif
- }
+ }
// Delete the process instance.
cmsysProcess_Delete(cp);