40,000 USDC
View results
Submission Details
Severity: gas

State variables should include comments

Summary

State variables should include comments

Vulnerability Details

Consider adding some comments on critical state variables to explain what they are supposed to do: this will help for future code reviews.

File: /src/Escrow.sol
//@audit i_price need comments
17: uint256 private immutable i_price;
//@audit i_buyer need comments
21: address private immutable i_buyer;
//@audit i_seller need comments
22: address private immutable i_seller;
//@audit i_arbiter need comments
23: address private immutable i_arbiter;
//@audit i_arbiterFee need comments
24: uint256 private immutable i_arbiterFee;
//@audit s_state need comments
26: State private s_state;

Link to code

Impact

Informational

Tools Used

Manual

Recommendations

add comments to critical state variables

Support

FAQs

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