Be pep8 compliant when forming display names

SergeySatskiy · Aug 1, 2018 · 9b55720 · 9b55720
1 parent 6ecb35e
commit 9b55720
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 6 deletions.
diff --git a/ChangeLog b/ChangeLog
@@ -1,3 +1,8 @@
+2018 Aug 01 Sergey Satskiy <[email protected]>
+
+* Release 3.0.3
+* Be pep8 compliant when display names are formed
+
 2017 Nov 16 Sergey Satskiy <[email protected]>
 
 * Release 3.0.2

diff --git a/cdmpyparser.py b/cdmpyparser.py
@@ -224,7 +224,7 @@ def getDisplayName(self):
         """Provides a name for display purpose"""
         displayName = self.name
         if self.arguments:
-            displayName += "( " + ", ".join(self.arguments) + " )"
+            displayName += "(" + ", ".join(self.arguments) + ")"
         return displayName
 
 
@@ -331,12 +331,10 @@ def getDisplayName(self):
         first = True
         for arg in self.arguments:
             if first:
-                displayName += " " + str(arg)
+                displayName += str(arg)
                 first = False
             else:
                 displayName += ", " + str(arg)
-        if self.arguments:
-            displayName += " "
         displayName += ")"
         if self.returnAnnotation is not None:
             displayName += ' -> ' + self.returnAnnotation
@@ -400,7 +398,7 @@ def getDisplayName(self):
         " Provides a name for display purpose "
         displayName = self.name
         if self.base:
-            displayName += "( " + ", ".join(self.base) + " )"
+            displayName += "(" + ", ".join(self.base) + ")"
         return displayName
 
 

diff --git a/cdmpyparserversion.py b/cdmpyparserversion.py
@@ -1 +1 @@
-version = '3.0.2'
+version = '3.0.3'