Omniscia Bonq Audit
Code Style
Code Style
During the manual portion of the audit, we identified 21 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 |
|---|---|---|---|
| BON-01C |  |  | Deprecated Numeric Representation |
| BON-02C | | | Repetitive Value Literal |
| ARB-01C | |  | Generic Typographic Mistakes |
| ARB-02C | | | Redundant Logical Check |
| ARB-03C | | | Variable Mutability Specifier (Immutable) |
| COM-01C | | | Variable Mutability Specifiers (Immutable) |
| CON-01C | | | Deprecated Numeric Representation |
| EXT-01C | | | Inefficient Loop Limit Evaluation |
| EXT-02C | | | Inefficient Ternary Operators |
| LIN-01C | |  | Generic Typographic Mistakes |
| LIN-02C | | | Inconsistent Logical Checks |
| LIN-03C | | | Inefficient Utilization of Data Entries |
| LIN-04C | | | Inefficient mapping Lookups |
| MIT-01C | | | Variable Mutability Specifier (Immutable) |
| PRI-01C | |  | Generic Typographic Mistake |
| PRI-02C | | | Inefficient Loop Limit Evaluation |
| PRI-03C | | | Inefficient Ternary Operators |
| TOK-01C | | | Repetitive Value Literal |
| TRE-01C | | | Variable Mutability Specifiers (Immutable) |
| TRO-01C | | | Ineffectual Value Assignment |
| TRO-02C | | | Variable Mutability Specifiers (Immutable) |