Beginner FriendlyFoundry
100 EXP
View results
Submission Details
Severity: low
Invalid

Function parameter should be 'calldata' not 'memory'

Summary

using memory keyword is inefficient in this case, as we are not manipulating the variable in any way and thus can use the cheaper, immutable, calldata.

Impact

Unnecessary gas costs, it is cheaper to use calldata

Recommendations

it is cheaper to use calldata, i.e :'function setPassword(string memory newPassword) '

Updates

Lead Judging Commences

inallhonesty Lead Judge
about 2 years ago
inallhonesty Lead Judge about 2 years ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

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