Submission Details
Severity:
Unreachable `PuppyRaffle::_isActivePlayer` Function
Summary
_isActivePlayer is declared with internal visibility modifier and is not call from any other function.
Impact
Waste of gas of deployment and unnecessary bigger contract size.
Tools Used
Manual
Recommendations
The function has a similar logic to getActivePlayerIndex making it possibly redundant. My recommendation is to remove the function from the contract.
Rewards breakdown
nSLOC
143This contest has a flat reward structure with the following payouts:
100 EXP
20 EXP
2 EXP
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.