Merge pull request #68 from OpenSimulationInterface/feature/travis-code-review

code analysis in Travis Ci (resolves #65)

Author: jdsika

.travis.yml

@@ -43,5 +43,6 @@ script:
     - cmake -D CMAKE_PREFIX_PATH:PATH=${DEPS_DIR}/protobuf/install ..
     - cmake --build .
     - cd ..
+    - python test_cases.py
     - python setup.py build
     - python setup.py sdist

doxygen_config.cmake.in

@@ -4,18 +4,18 @@ PROJECT_NUMBER         = @VERSION_MAJOR@.@VERSION_MINOR@.@VERSION_PATCH@
 
 #Set input and output directories 
 INPUT                  = "@CMAKE_CURRENT_SOURCE_DIR@"
-INPUT       	      += "@CMAKE_CURRENT_SOURCE_DIR@/README.md"
+INPUT                 += "@CMAKE_CURRENT_SOURCE_DIR@/README.md"
 
 USE_MDFILE_AS_MAINPAGE = "@CMAKE_CURRENT_SOURCE_DIR@/README.md"
 
 OUTPUT_DIRECTORY       = "@CMAKE_CURRENT_SOURCE_DIR@/doc"
-IMAGE_PATH	       = "@CMAKE_CURRENT_SOURCE_DIR@/doc/images"
+IMAGE_PATH             = "@CMAKE_CURRENT_SOURCE_DIR@/doc/images"
 
 # There is no standard configuration for .proto files documentation.
 # A Doxygen filter for .proto files has to be added under the directory doc/.
 JAVADOC_AUTOBRIEF      = NO
 EXTENSION_MAPPING      = proto=C
 FILE_PATTERNS          = *.proto
 INPUT_FILTER           = "python @FILTER_PROTO2CPP_PY_PATH@/proto2cpp.py"
-HAVE_DOT	       = YES
-			
+HAVE_DOT               = YES
+            

test_cases.py

@@ -0,0 +1,23 @@
+import sys
+from glob import *
+
+state = 0
+
+for file in glob("*.*"):
+    if(file != "test_cases.py"):
+        with open(file, "rt") as fin:
+            i = 0
+            for line in fin:
+                i = i + 1
+
+                # Test case 1 is checking if there are illegal tabulators in the code
+                if line.find("\t") != -1:
+                    print(file + " in line " + str(i) + ": not permitted tab found")
+                    state = 1
+
+                # Test case 2 is checking if there are more than the two allowed '/'
+                if line.find("///") != -1:
+                    print(file + " in line " + str(i) + ": not permitted use of '///' ")
+                    state = 1
+					
+sys.exit(state)