Omniscia Stakewise Audit

Code Style

Code Style

During the manual portion of the audit, we identified 15 optimizations that can be applied to the codebase that will decrease the gas-cost associated with the execution of a particular function and generally ensure that the project complies with the latest best practices and standards in Solidity.

Additionally, this section of the audit contains any opinionated adjustments we believe the code should make to make it more legible as well as truer to its purpose.

These optimizations are enumerated below:

IDSeverityAddressedTitle
ERP-01CInformationalNoSub-Optimal EIP-712 Implementation
ERP-02CInformationalNoUnoptimized Variable Mutability
ERC-01CInformationalNoDeprecated Maximum Representation Style
ORA-01CInformationalYesInefficient Block Number Comparison
ORA-02CInformationalYesMultiple Top-Level Declarations
ORA-03CInformationalNoRedundant Visibility Specifier
ORA-04CInformationalYesUndocumented Consortium Level
OPE-01CInformationalNoRedundant Visibility Specifier
OPU-01CInformationalNoRedundant Visibility Specifier
POO-01CInformationalYesUndocumented Value Literal
PVS-01CInformationalNoRedundant Root Validations
RET-01CInformationalYesDuplicate Event Emittance & Storage Write
ROL-01CInformationalYesUnspecified Numerical Accuracy
SET-01CInformationalNoIncorrect Gas Optimization
SET-02CInformationalNoPotential XOR Optimization