search

code-423n4 / 2022-08-foundation-findings

0 stars 0 forks source link

QA Report #288

Closed code423n4 closed 1 year ago

code423n4 commented 1 year ago

See the markdown file with the details of this report here.

HardlyDifficult commented 1 year ago

It is unnecesary to iniliaze variables in for loops

Disagree. This is usually submitted as a potential gas optimization. From a QA perspective I believe this hurts readability since that is not normally how for loops are written in any language. From the gas perspective, testing shows that this optimization is no longer effective with the latest versions of Solidity.