20,000 USDC
View results
Submission Details
Severity: gas
Valid

Spelling/Errors Organization

Summary

Structs.sol

Line #75

- /// @notice the virtual balance based on the multipier
+ /// @notice the virtual balance based on the multiplier

Perhaps a suggestion to order the Errors.sol in the utils folder alphabetically. After searching I could not find a best practice. I do understand however, grouping them together via functional relationship. To me however, alphabetically is easier to read and a sorted list can make version control diffs cleaner when adding or removing errors in the future. You could similarly alphabetized the Structs.sol file.

As is:

// SPDX-License-Identifier: MIT
pragma solidity ^0.8.19;
error PoolConfig();
error LoanTooSmall();
error LoanTooLarge();
error RatioTooHigh();
error FeeTooHigh();
error TokenMismatch();
error Unauthorized();
error AuctionStarted();
error AuctionNotStarted();
error AuctionEnded();
error AuctionNotEnded();
error RateTooHigh();
error PoolTooSmall();
error ZeroCollateral();
error AuctionTooShort();

Alphabetical:

// SPDX-License-Identifier: MIT
pragma solidity ^0.8.19;
error AuctionEnded();
error AuctionNotEnded();
error AuctionNotStarted();
error AuctionStarted();
error AuctionTooShort();
error FeeTooHigh();
error LoanTooLarge();
error LoanTooSmall();
error PoolConfig();
error PoolTooSmall();
error RateTooHigh();
error RatioTooHigh();
error TokenMismatch();
error Unauthorized();
error ZeroCollateral();

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.