Unused argument 'error'
11 raise error(msg)
12
13
14def preprocess(seq, error=_handle_error):15 for tok, val in list(seq):
16 if tok == IN:
17 negate = val[0] == (NEGATE, None)
Description
An unused argument can lead to confusions. It should be removed. If this variable is necessary, name the variable _ or start the name with unused or _unused.
Bad practice
def square(x, y=1):
return x * x
class MySubClass(MyClass):
def __init__(self, number):
self.value = 42 # argument `number` remains unused
Preferred:
def square(x):
return x * x
class MySubClass(MyClass):
def __init__(self, _):
self.value = 42