Use immutable arg rather mutable arg
Passing mutable objects as default args is a known Python pitfall. We'd better avoid this. This commit changes mutable default args with None. Change-Id: I78323383c8b6a9a516a2a217303124870822418b
This commit is contained in:
@@ -45,7 +45,8 @@ def arg(*args, **kwargs):
|
||||
return _decorator
|
||||
|
||||
|
||||
def schema_args(schema_getter, omit=[]):
|
||||
def schema_args(schema_getter, omit=None):
|
||||
omit = omit or []
|
||||
typemap = {
|
||||
'string': str,
|
||||
'integer': int,
|
||||
@@ -104,7 +105,8 @@ def pretty_choice_list(l):
|
||||
return ', '.join("'%s'" % i for i in l)
|
||||
|
||||
|
||||
def print_list(objs, fields, formatters={}):
|
||||
def print_list(objs, fields, formatters=None):
|
||||
formatters = formatters or {}
|
||||
pt = prettytable.PrettyTable([f for f in fields], caching=False)
|
||||
pt.align = 'l'
|
||||
|
||||
|
Reference in New Issue
Block a user