Omniscia Olympus DAO Audit
StandardBondingCalculator Code Style Findings
StandardBondingCalculator Code Style Findings
SBC-01C: Inexistent Error Message
| Type | Severity | Location |
|---|---|---|
| Code Style | Informational | StandardBondingCalculator.sol:L26 |
Description:
The linked require check contains no descriptive error message.
Example:
contracts/StandardBondingCalculator.sol
26require( _OHM != address(0) );Recommendation:
We advise one to be set so to aid in the debugging of the application and to also enable more accurate validation of the require condition's purpose.
Alleviation:
Error messages were introduced in all linked require checks.
SBC-02C: Inexistent Variable Visibility Specifier
| Type | Severity | Location |
|---|---|---|
| Code Style | Informational | StandardBondingCalculator.sol:L23 |
Description:
The linked variable has no visibility specifier explicitly set.
Example:
contracts/StandardBondingCalculator.sol
23IERC20 immutable OHM;Recommendation:
We advise one to be set so to avoid potential compilation discrepancies in the future as the current compiler behaviour is to assign a specifier automatically.
Alleviation:
The internal visibility specifier was properly introduced to the linked variable.