Alchemix Transmuter
Alchemix
DeFiFoundrySolidity
16,653 OP
View results
Previous Next
Submission Details
Severity: low
Invalid

Typographical Errors

player

Summary

Typographical errors in variable names can create an impression of insufficient attention to detail during the development process. In the StrategyOp.sol, StrategyArb.sol, and StrategyMainnet.sol contracts, the same function contains a misspelled variable name:

function _freeFunds(uint256 _amount) internal override {
uint256 totalAvailabe = transmuter.getUnexchangedBalance(address(this));
if (_amount > totalAvailabe) {
transmuter.withdraw(totalAvailabe, address(this));
} else {
transmuter.withdraw(_amount, address(this));
}
}

The variable totalAvailabe is misspelled and should be corrected for improved clarity and professionalism.

Impact

Misspelled variable names may reduce code readability and lead to misunderstandings, especially for external auditors and contributors. Additionally, such errors may give the impression of carelessness in the development process.

Tools Used

Manual review.

Recommendations

Rename the variable totalAvailabe to totalAvailable to improve clarity and maintain consistency in variable naming conventions across the codebase.

Updates

Appeal created

inallhonesty Lead Judge 6 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity
inallhonesty Lead Judge 6 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

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