DaiGoldAuction::startAuction
function uses storage
instead of memory
which will cost more gas than necessary
The DaiGoldAuction::startAuction
function uses the storage
keyword to read data from the blockchain
and did not cause any changes to the state variable within that function.
Because the DaiGoldAuction::startAuction
function uses the storage
keyword to read data from the blockchain without causing any changes to the state variable within that function, it attracts unnecessary gas. It would be cheaper to use the memory
keyword in this scenaro since the function is not causing any state changes to the auctionConfig
variable.
Manual review
The section of DaiGoldAuction::startAuction
function that reads the auctionConfig
variable from the blockchain can be rearranged as follows
The contest is live. Earn rewards by submitting a finding.
This is your time to appeal against judgements on your submissions.
Appeals are being carefully reviewed by our judges.