pre-commit with auto fixing
authorEduardo Ramos Testillano <eduardo.ramos.testillano@gmail.com>
Tue, 25 Jun 2013 01:21:36 +0000 (03:21 +0200)
committerEduardo Ramos Testillano <eduardo.ramos.testillano@gmail.com>
Tue, 25 Jun 2013 01:21:36 +0000 (03:21 +0200)
pre-commit.sh
source/core/util/Environment.cpp

index 2771db8..e9ded58 100755 (executable)
@@ -93,13 +93,18 @@ test_style () {
   $ASTYLE -a -f -p -o -O -c -s2 -U -x --mode=c < $file > $newfile 2>>/dev/null
   diff "${file}" "${newfile}"
   r=$?
-  rm "${newfile}"
   if [ $r != 0 ] ; then
     echo "Code style error in '$file', please fix before commiting."
-    echo "To autofix, create a hidden file named './.fix_style' on suite root."
-pwd
-echo xxxx
-    exit 1
+    if [ -f "./.fix_style" ]; then
+      echo "Auto-fixing code style..."
+      mv $newfile $file
+    else
+      echo "To autofix, create a hidden file named './.fix_style' on suite root."
+      rm $newfile
+      exit 1
+    fi
+  else
+    rm $newfile
   fi
 }
 
index c00d9a5..83ed2f8 100644 (file)
@@ -112,7 +112,7 @@ std::string Environment::getValue(const char* variableName, bool exceptionIfMiss
 
   std::map<std::string, std::string>::const_iterator it = a_managedVars.find(variableName);
 
-  if   (it == a_managedVars.end()) {
+  if(it == a_managedVars.end()) {
     std::string msg = "The variable '";
     msg += variableName;
     msg += "' is not defined in the environment.";