40,000 USDC
View results
Submission Details
Severity: gas

Lines are too long.

Summary

The following lines are too long to be readable.

Vulnerability Details

Usually lines in source code are limited to 80 characters. Today's screens are much larger so it's reasonable to stretch this in some cases. The solidity style guide recommends a maximum line length of 120 characters, so the lines below should be split when they reach that length.

There are 3 instances of this issue.

File: 2023-07-escrow/src/Escrow.sol

4: // Inspired by `BillOfSaleERC20` contract: https://github.com/open-esq/Digital-Organization-Designs/blob/master/Finance/BillofSaleERC20.sol
28: /// @dev Sets the Escrow transaction values for `price`, `tokenContract`, `buyer`, `seller`, `arbiter`, `arbiterFee`. All of
30: /// @dev Funds should be sent to this address prior to its deployment, via create2. The constructor checks that the tokens have

Impact

Tools Used

Manual code review.

Recommendations

Reduce number of characters per line to improve readability.

Support

FAQs

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