Omniscia Stakewise Audit
OwnablePausable Code Style Findings
OwnablePausable Code Style Findings
OPE-01C: Redundant Visibility Specifier
| Type | Severity | Location |
|---|---|---|
| Gas Optimization | Informational | OwnablePausable.sol:L16 |
Description:
The linked variable is meant to be used as an internally accessible constant and has no use outside of the contract as it represents a static value.
Example:
contracts/presets/OwnablePausable.sol
16bytes32 public constant PAUSER_ROLE = keccak256("PAUSER_ROLE");Recommendation:
We advise it to be set to either internal or private to reduce the bytecode size of the contract.
Alleviation:
The Stakewise team stated that they prefer to retain the current visibility in place to ensure non-technically attuned persons can still read the status of users in the system when using basic tools such as Etherscan.