Omniscia DAFI Audit
Code Style
Code Style
During the manual portion of the audit, we identified 19 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:
ID | Severity | Addressed | Title |
---|---|---|---|
NDD-01C | Informational | Yes | Convoluted Code |
NDD-02C | Informational | Yes | Inexistent Visibility Specifiers |
NDD-03C | Informational | Yes | Variable Mutability Specifier |
PFS-01C | Informational | Yes | Variable Mutability Specifier |
REE-01C | Informational | No | Inexistent Visibility Specifiers |
REE-02C | Informational | Yes | Redundant Getter Invocation |
REE-03C | Informational | Yes | Redundant Return Value |
SDE-01C | Informational | Yes | Duplication of Code |
SDE-02C | Informational | No | Inexistent Visibility Specifiers |
SDE-03C | Informational | Yes | Literal Boolean Comparison |
SDE-04C | Informational | Yes | Redundant Function Implementation |
SDE-05C | Informational | Yes | Redundant Visibility Specifier |
SMV-01C | Informational | Yes | Code Inconsistency |
SMV-02C | Informational | No | Inexistent Visibility Specifiers |
SMV-03C | Informational | Yes | Redundant Logic Paths |
SMV-04C | Informational | Yes | Variable Mutability Specifier |
TVL-01C | Informational | Yes | Inexistent Visibility Specifier |
TPL-01C | Informational | No | Type Optimization |
TPL-02C | Informational | Yes | Variable Mutability Specifier |