espresso-sequencer icon indicating copy to clipboard operation
espresso-sequencer copied to clipboard

I06: Inefficient use of memory for read-only arguments

Open philippecamacho opened this issue 1 year ago • 3 comments

Affected Code: PlonkVerifier.sol (line 91)

Summary: The verify function takes three arguments all of which are read-only.

Suggestion: We recommend using calldata for proof, publicInput, and verifyingKey arguments and making the necessary changes to the rest of the codebase to support this update.

philippecamacho avatar Jul 18 '24 15:07 philippecamacho