Merge branch 'master' into 'master'

fix some issues in Codacy See merge request c4g/gltf/libgltf!1
code4game · Mar 13, 2018 · 9283818 · 9283818
2 parents e2c1c41 + e50e215
commit 9283818
Show file tree

Hide file tree

Showing 5 changed files with 39 additions and 34 deletions.
diff --git a/tools/jsonschematoc11/c11types/c11typebool.py b/tools/jsonschematoc11/c11types/c11typebool.py
@@ -15,19 +15,20 @@ def codeDefaultValue(cls, schemaDefaultValue):
         return u'false'
 
     @classmethod
-    def codeDefaultValueArray(cls, schemaDefaultValue):
-        if schemaDefaultValue is None\
-            or not isinstance(schemaDefaultValue, list)\
-            or len(schemaDefaultValue) <= 0:
+    def codeDefaultValueArray(cls, schemaDefaultValues):
+        if schemaDefaultValues is None\
+            or not isinstance(schemaDefaultValues, list)\
+            or len(schemaDefaultValues) <= 0:
             return u''
-        codeDefauleValue = u''
-        for i in range(0, len(schemaDefaultValue)):
-            if schemaDefaultValue[i] == True:
-                codeDefauleValue = codeDefauleValue + u'true, '
+        code_default_value = u''
+        for schema_value in schemaDefaultValues:
+            if len(code_default_value) > 0:
+                code_default_value = code_default_value + u', '
+            if schema_value == True:
+                code_default_value = code_default_value + u'true'
             else:
-                codeDefauleValue = codeDefauleValue + u'false, '
-        codeDefauleValue = codeDefauleValue[0:len(codeDefauleValue) - 2]
-        return u'{ %s }' % codeDefauleValue
+                code_default_value = code_default_value + u'false'
+        return u'{ %s }' % code_default_value
 
     @classmethod
     def codeJsonCheck(cls):

diff --git a/tools/jsonschematoc11/c11types/c11typeinteger.py b/tools/jsonschematoc11/c11types/c11typeinteger.py
@@ -16,14 +16,15 @@ def codeDefaultValue(cls, schemaDefaultValue):
         return u'0'
 
     @classmethod
-    def codeDefaultValueArray(cls, schemaDefaultValue):
-        if schemaDefaultValue is None or isinstance(schemaDefaultValue, list) or len(schemaDefaultValue) <= 0:
+    def codeDefaultValueArray(cls, schemaDefaultValues):
+        if schemaDefaultValues is None or isinstance(schemaDefaultValues, list) or len(schemaDefaultValues) <= 0:
             return u''
-        codeDefauleValue = u''
-        for i in range(0, len(schemaDefaultValue)):
-            codeDefauleValue = codeDefauleValue + u'%d, ' % schemaDefaultValue[i]
-        codeDefauleValue = codeDefauleValue[0:len(codeDefauleValue) - 2]
-        return u'{ %s }' % codeDefauleValue
+        code_default_value = u''
+        for schema_value in schemaDefaultValues:
+            if len(code_default_value) > 0:
+                code_default_value = code_default_value + u', '
+            code_default_value = code_default_value + u'%d' % schema_value
+        return u'{ %s }' % code_default_value
 
     @classmethod
     def codeJsonCheck(cls):

diff --git a/tools/jsonschematoc11/c11types/c11typenumber.py b/tools/jsonschematoc11/c11types/c11typenumber.py
@@ -16,16 +16,17 @@ def codeDefaultValue(cls, schemaDefaultValue):
         return u'0.0f'
 
     @classmethod
-    def codeDefaultValueArray(cls, schemaDefaultValue):
-        if schemaDefaultValue is None\
-            or not isinstance(schemaDefaultValue, list)\
-            or len(schemaDefaultValue) <= 0:
+    def codeDefaultValueArray(cls, schemaDefaultValues):
+        if schemaDefaultValues is None\
+            or not isinstance(schemaDefaultValues, list)\
+            or len(schemaDefaultValues) <= 0:
             return u''
-        codeDefauleValue = u''
-        for i in range(0, len(schemaDefaultValue)):
-            codeDefauleValue = codeDefauleValue + u'%ff, ' % schemaDefaultValue[i]
-        codeDefauleValue = codeDefauleValue[0:len(codeDefauleValue) - 2]
-        return u'{ %s }' % codeDefauleValue
+        code_default_value = u''
+        for schema_value in schemaDefaultValues:
+            if len(code_default_value) > 0:
+                code_default_value = code_default_value + u', '
+            code_default_value = code_default_value + u'%ff' % schema_value
+        return u'{ %s }' % code_default_value
 
     @classmethod
     def codeJsonCheck(cls):

diff --git a/tools/jsonschematoc11/c11types/c11typestring.py b/tools/jsonschematoc11/c11types/c11typestring.py
@@ -19,14 +19,16 @@ def codeDefaultValue(cls, schemaDefaultValue):
         return u'GLTFTEXT("")'
 
     @classmethod
-    def codeDefaultValueArray(self, schemaDefaultValue):
-        if schemaDefaultValue is None\
-            or not isinstance(schemaDefaultValue, list)\
-            or len(schemaDefaultValue) <= 0:
+    def codeDefaultValueArray(self, schemaDefaultValues):
+        if schemaDefaultValues is None\
+            or not isinstance(schemaDefaultValues, list)\
+            or len(schemaDefaultValues) <= 0:
             return u''
         code_default_value = u''
-        for i in range(0, len(schemaDefaultValue)):
-            code_default_value = code_default_value + u'GLTFTEXT("%s"), ' % schemaDefaultValue[i]
+        for schema_value in schemaDefaultValues:
+            if len(code_default_value) > 0:
+                code_default_value = code_default_value + u', '
+            code_default_value = code_default_value + u'GLTFTEXT("%s")' % schema_value
         code_default_value = code_default_value[0:len(code_default_value) - 2]
         return u'{ %s }' % code_default_value
 

diff --git a/tools/jsonschematoc11/c11types/c11typestruct.py b/tools/jsonschematoc11/c11types/c11typestruct.py
@@ -215,7 +215,7 @@ def codeSource(self, codeTypeNames):
             codeLines.append(u'{')
             codeLines.append(u'    return %sValue;' % (self.c11Type.codeTypeName()))
             codeLines.append(u'}')
-            
+
         if self.manualCodeSourcesFunction is not None and len(self.manualCodeSourcesFunction) > 0:
             codeLines.append(u'')
             codeLines.append(u'// Manual code lines')