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.

1 files changed, 48 insertions, 88 deletions

diff --git a/Source/cmSetSourceFilesPropertiesCommand.cxx b/Source/cmSetSourceFilesPropertiesCommand.cxx
index 19c681944c..976f3c8ee2 100644
--- a/Source/cmSetSourceFilesPropertiesCommand.cxx
+++ b/Source/cmSetSourceFilesPropertiesCommand.cxx
@@ -14,157 +14,117 @@
 #include "cmSourceFile.h"
 
 // cmSetSourceFilesPropertiesCommand
-bool cmSetSourceFilesPropertiesCommand
-::InitialPass(std::vector<std::string> const& args, cmExecutionStatus &)
+bool cmSetSourceFilesPropertiesCommand::InitialPass(
+  std::vector<std::string> const& args, cmExecutionStatus&)
 {
-  if(args.size() < 2 )
-    {
+  if (args.size() < 2) {
     this->SetError("called with incorrect number of arguments");
     return false;
-    }
+  }
 
   // break the arguments into source file names and properties
   int numFiles = 0;
   std::vector<std::string>::const_iterator j;
   j = args.begin();
   // old style allows for specifier before PROPERTIES keyword
-  while (j != args.end() &&
-         *j != "ABSTRACT" &&
-         *j != "WRAP_EXCLUDE" &&
-         *j != "GENERATED" &&
-         *j != "COMPILE_FLAGS" &&
-         *j != "OBJECT_DEPENDS" &&
-         *j != "PROPERTIES")
-    {
+  while (j != args.end() && *j != "ABSTRACT" && *j != "WRAP_EXCLUDE" &&
+         *j != "GENERATED" && *j != "COMPILE_FLAGS" &&
+         *j != "OBJECT_DEPENDS" && *j != "PROPERTIES") {
     numFiles++;
     ++j;
-    }
+  }
 
   // now call the worker function
   std::string errors;
-  bool ret =
-    cmSetSourceFilesPropertiesCommand
-    ::RunCommand(this->Makefile,
-                 args.begin(),
-                 args.begin() + numFiles,
-                 args.begin() + numFiles,
-                 args.end(), errors);
-  if (!ret)
-    {
+  bool ret = cmSetSourceFilesPropertiesCommand::RunCommand(
+    this->Makefile, args.begin(), args.begin() + numFiles,
+    args.begin() + numFiles, args.end(), errors);
+  if (!ret) {
     this->SetError(errors);
-    }
+  }
   return ret;
 }
 
-bool cmSetSourceFilesPropertiesCommand
-::RunCommand(cmMakefile *mf,
-             std::vector<std::string>::const_iterator filebeg,
-             std::vector<std::string>::const_iterator fileend,
-             std::vector<std::string>::const_iterator propbeg,
-             std::vector<std::string>::const_iterator propend,
-             std::string &errors)
+bool cmSetSourceFilesPropertiesCommand::RunCommand(
+  cmMakefile* mf, std::vector<std::string>::const_iterator filebeg,
+  std::vector<std::string>::const_iterator fileend,
+  std::vector<std::string>::const_iterator propbeg,
+  std::vector<std::string>::const_iterator propend, std::string& errors)
 {
   std::vector<std::string> propertyPairs;
   bool generated = false;
   std::vector<std::string>::const_iterator j;
   // build the property pairs
-  for(j= propbeg; j != propend;++j)
-    {
+  for (j = propbeg; j != propend; ++j) {
     // old style allows for specifier before PROPERTIES keyword
-    if(*j == "ABSTRACT")
-      {
+    if (*j == "ABSTRACT") {
       propertyPairs.push_back("ABSTRACT");
       propertyPairs.push_back("1");
-      }
-    else if(*j == "WRAP_EXCLUDE")
-      {
+    } else if (*j == "WRAP_EXCLUDE") {
       propertyPairs.push_back("WRAP_EXCLUDE");
       propertyPairs.push_back("1");
-      }
-    else if(*j == "GENERATED")
-      {
+    } else if (*j == "GENERATED") {
       generated = true;
       propertyPairs.push_back("GENERATED");
       propertyPairs.push_back("1");
-      }
-    else if(*j == "COMPILE_FLAGS")
-      {
+    } else if (*j == "COMPILE_FLAGS") {
       propertyPairs.push_back("COMPILE_FLAGS");
       ++j;
-      if(j == propend)
-        {
+      if (j == propend) {
         errors = "called with incorrect number of arguments "
-          "COMPILE_FLAGS with no flags";
+                 "COMPILE_FLAGS with no flags";
         return false;
-        }
-      propertyPairs.push_back(*j);
       }
-    else if(*j == "OBJECT_DEPENDS")
-      {
+      propertyPairs.push_back(*j);
+    } else if (*j == "OBJECT_DEPENDS") {
       propertyPairs.push_back("OBJECT_DEPENDS");
       ++j;
-      if(j == propend)
-        {
+      if (j == propend) {
         errors = "called with incorrect number of arguments "
-          "OBJECT_DEPENDS with no dependencies";
+                 "OBJECT_DEPENDS with no dependencies";
         return false;
-        }
-      propertyPairs.push_back(*j);
       }
-    else if(*j == "PROPERTIES")
-      {
+      propertyPairs.push_back(*j);
+    } else if (*j == "PROPERTIES") {
       // now loop through the rest of the arguments, new style
       ++j;
-      while (j != propend)
-        {
+      while (j != propend) {
         propertyPairs.push_back(*j);
-        if(*j == "GENERATED")
-          {
+        if (*j == "GENERATED") {
           ++j;
-          if(j != propend && cmSystemTools::IsOn(j->c_str()))
-            {
+          if (j != propend && cmSystemTools::IsOn(j->c_str())) {
             generated = true;
-            }
           }
-        else
-          {
+        } else {
           ++j;
-          }
-        if(j == propend)
-          {
+        }
+        if (j == propend) {
           errors = "called with incorrect number of arguments.";
           return false;
-          }
+        }
         propertyPairs.push_back(*j);
         ++j;
-        }
+      }
       // break out of the loop because j is already == end
       break;
-      }
-    else
-      {
+    } else {
       errors = "called with illegal arguments, maybe missing a "
-        "PROPERTIES specifier?";
+               "PROPERTIES specifier?";
       return false;
-      }
     }
+  }
 
   // now loop over all the files
-  for(j= filebeg; j != fileend;++j)
-    {
+  for (j = filebeg; j != fileend; ++j) {
     // get the source file
-    cmSourceFile* sf =
-      mf->GetOrCreateSource(*j, generated);
-    if(sf)
-      {
+    cmSourceFile* sf = mf->GetOrCreateSource(*j, generated);
+    if (sf) {
       // now loop through all the props and set them
       unsigned int k;
-      for (k = 0; k < propertyPairs.size(); k = k + 2)
-        {
-        sf->SetProperty(propertyPairs[k],propertyPairs[k+1].c_str());
-        }
+      for (k = 0; k < propertyPairs.size(); k = k + 2) {
+        sf->SetProperty(propertyPairs[k], propertyPairs[k + 1].c_str());
       }
     }
+  }
   return true;
 }
-