Gets the short identifier of the detector, used in analyzer warnings.
The unique identifier of the detector.
Gets the kind of the detector.
Defines the behavior of warnings generated by this detector when working with multiple projects within a single Tact configuration.
Here are the available options:
"union"
Leave this value if you don't care about warnings generated in other projects."intersect"
If the warning is generated for some source location of the imported file,
it should be generated by each of the projects. Example: Constants from an
imported file should not be reported iff they are unused in all the projects,
so you need "intersect".Checks whether this detector needs the Soufflé binary to be executed.
Executes the detector's logic to check for issues within the provided compilation unit.
The compilation unit to be analyzed.
List of warnings has highlighted by this detector.
Protected
makeProtected
skip
Detects method calls where an argument is always the same constant or contract field, suggesting removal of the redundant parameter for gas optimization.
Why is this bad?
Passing the same value every damn time wastes gas and clutters the code. If an argument is always
self.a
or a literal like42
, just hardcode it inside the method.What it checks
self.x
,self.y
, etc.).true
,42
,"fixed_string"
).Example
Use instead: