diff options
author | Kitware Robot <kwrobot@kitware.com> | 2016-05-16 14:34:04 (GMT) |
---|---|---|
committer | Brad King <brad.king@kitware.com> | 2016-05-16 20:05:19 (GMT) |
commit | d9fd2f5402eeaa345691313658e02b51038f570b (patch) | |
tree | dca71b9a7e267f4c6300da3eb770415381726785 /Source/cmExecProgramCommand.cxx | |
parent | 82df6deaafb36cbbfd450202bb20b320f637751a (diff) | |
download | CMake-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.cxx | 221 |
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); |