Skip to content

Commit 9b55720

Browse files
committed
Be pep8 compliant when forming display names
1 parent 6ecb35e commit 9b55720

File tree

3 files changed

+9
-6
lines changed

3 files changed

+9
-6
lines changed

ChangeLog

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,8 @@
1+
2018 Aug 01 Sergey Satskiy <sergey.satskiy@gmail.com>
2+
3+
* Release 3.0.3
4+
* Be pep8 compliant when display names are formed
5+
16
2017 Nov 16 Sergey Satskiy <sergey.satskiy@gmail.com>
27

38
* Release 3.0.2

cdmpyparser.py

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -224,7 +224,7 @@ def getDisplayName(self):
224224
"""Provides a name for display purpose"""
225225
displayName = self.name
226226
if self.arguments:
227-
displayName += "( " + ", ".join(self.arguments) + " )"
227+
displayName += "(" + ", ".join(self.arguments) + ")"
228228
return displayName
229229

230230

@@ -331,12 +331,10 @@ def getDisplayName(self):
331331
first = True
332332
for arg in self.arguments:
333333
if first:
334-
displayName += " " + str(arg)
334+
displayName += str(arg)
335335
first = False
336336
else:
337337
displayName += ", " + str(arg)
338-
if self.arguments:
339-
displayName += " "
340338
displayName += ")"
341339
if self.returnAnnotation is not None:
342340
displayName += ' -> ' + self.returnAnnotation
@@ -400,7 +398,7 @@ def getDisplayName(self):
400398
" Provides a name for display purpose "
401399
displayName = self.name
402400
if self.base:
403-
displayName += "( " + ", ".join(self.base) + " )"
401+
displayName += "(" + ", ".join(self.base) + ")"
404402
return displayName
405403

406404

cdmpyparserversion.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
version = '3.0.2'
1+
version = '3.0.3'

0 commit comments

Comments
 (0)