Fixes a false alarm about converting the empty string parameter name to mixedCase (#260).

pull/238/head
rajeevgopalakrishna 6 years ago
parent 85c3133634
commit 10e31b0893
  1. 3
      slither/detectors/naming_convention/naming_convention.py

@ -104,6 +104,9 @@ Solidity defines a [naming convention](https://solidity.readthedocs.io/en/v0.4.2
results.append(json) results.append(json)
for argument in func.parameters: for argument in func.parameters:
# Ignore parameter names that are not specified i.e. empty strings
if argument.name == "":
continue
if argument in func.variables_read_or_written: if argument in func.variables_read_or_written:
correct_naming = self.is_mixed_case(argument.name) correct_naming = self.is_mixed_case(argument.name)
else: else:

Loading…
Cancel
Save