Submission Details
Severity:
Named parameters mapping
Summary
Since their introduction in Solidity 0.8.18, it's advisable to use named parameters in mapping types to improve code and ABI readability . There are 4 instances of mappings that lack named parameters:
1 in Lender.sol
--mapping(bytes32 => Pool) public pools;
++mapping(bytes32 poolId => Pool pool) public pools;
3 in Stakable.sol
--mapping(address => uint256) public supplyIndex;
--mapping(address => uint256) public balances;
--mapping(address => uint256) public claimable;
++mapping(address user => uint256 index) public supplyIndex;
++mapping(address user => uint256 balance) public balances;
++mapping(address user => uint256 claimableRewards) public claimable;
Prize pool breakdown
Total prize
20,000 USDC
nSLOC
70628
USDC / LOC
18,000 USDC
2,000 USDC
Live
The contest is live. Earn rewards by submitting a finding.
Appeals
This is your time to appeal against judgements on your submissions.
Appeals Review
Appeals are being carefully reviewed by our judges.