Omniscia Evergon Labs Audit

Code Style

Code Style

During the manual portion of the audit, we identified 83 optimizations that can be applied to the codebase that will decrease the operational 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
ACS-01CImproper Role Definitions
ACS-02CInefficient mapping Lookups
ACS-03CNon-Standard Storage Slot Definition
ACS-04CRedundant Logical Structures
AUS-01CInefficient mapping Lookups
AUS-02CNon-Standard Storage Slot Definition
BAR-01CGeneric Typographic Mistake
BAR-02CRedundant Named Function Argument
BFF-01CGeneric Typographic Mistake
BFD-01CNon-Standard Storage Slot Definition
BFD-02CRepetitive Value Literal
BSE-01CNon-Standard Storage Slot Definition
CAT-01CNon-Standard Storage Slot Definition
CFM-01CInefficient Code Structure
CFM-02CNon-Standard Storage Slot Definition
CFG-01CIneffectual Usage of Safe Arithmetics
CFG-02CNon-Standard Storage Slot Definition
CFS-01CInefficient mapping Lookups
CFN-01CInefficient mapping Lookups
COB-01CRedundant Named Function Argument
COM-01CRedundant Named Function Argument
DBF-01CNon-Standard Storage Slot Definition
ERR-01CNon-Standard Storage Slot Definition
EFS-01CInconsistent State Definitions
EFS-02CNon-Standard Storage Slot Definition
FBF-01CNon-Standard Storage Slot Definition
FDM-01CMisleading Function Names
FDC-01CNon-Standard Storage Slot Definition
FDC-02CRedundant Conditional Structure
FIA-01CInefficient Data Management
FIA-02CMisleading Function Name
FIA-03CNon-Standard Storage Slot Definition
FIA-04CRedundant Yielding of Buyback Amount
FIA-05CRepetitive Value Literal
FAF-01CNon-Standard Storage Slot Definition
FRF-01CInefficient Logic Structures
FRF-02CNon-Standard Storage Slot Definition
FAS-01CNon-Standard Storage Slot Definition
GSE-01CNon-Standard Storage Slot Definition
LOS-01CCombination of Logical Clauses
LOS-02CIneffectual Usage of Safe Arithmetics
LOS-03CMisleading Codebase Notions
LOS-04CNon-Standard Storage Slot Definition
LOS-05CRepetitive Value Literal
LRF-01CNon-Standard Storage Slot Definition
MFF-01CNon-Standard Storage Slot Definition
MFF-02CRepetitive Value Literal
MSF-01CNon-Standard Storage Slot Definition
NFP-01CNon-Standard Storage Slot Definition
NFP-02CUnused Library Implementation
OBF-01CNon-Standard Storage Slot Definition
PPF-01CInefficient mapping Lookups
PPF-02CNon-Standard Storage Slot Definition
PDM-01CGeneric Typographic Mistake
PDS-01CNon-Standard Storage Slot Definition
PRF-01CNon-Standard Storage Slot Definition
PTA-01CNon-Standard Storage Slot Definition
RAE-01CNon-Standard Storage Slot Definition
RAF-01CNon-Standard Storage Slot Definition
RAG-01CRedundant Named Function Argument
RBF-01CRedundant Named Function Argument
SFF-01CNon-Standard Storage Slot Definition
SFF-02CRepetitive Value Literal
SSF-01CNon-Standard Storage Slot Definition
SSA-01CNon-Standard Storage Slot Definition
SSC-01CNon-Standard Storage Slot Definition
SSG-01CNon-Standard Storage Slot Definition
SSE-01CNon-Standard Storage Slot Definition
SSO-01CNon-Standard Storage Slot Definition
SSS-01CNon-Standard Storage Slot Definition
SFS-01CNon-Standard Storage Slot Definition
TFA-01CRedundant Named Function Arguments
TFO-01CRedundant Named Function Arguments
TBT-01CRedundant Initialization Mechanism
TBF-01CNon-Standard Storage Slot Definition
TCA-01CInefficient mapping Lookups
TCA-02CNon-Standard Storage Slot Definition
TCA-03CRedundant Conditional Structure
UPV-01CRedundant Named Function Argument
UPF-01CNon-Standard Storage Slot Definition
WCS-01CInefficient Code Structure
WCS-02CNon-Standard Storage Slot Definition
WAS-01CNon-Standard Storage Slot Definition