code4rena-2022-08-nounsdao-n08
[N‑08] Lines are too long
Summary
코드 한 라인에 글자가 너무 많다고 지적했다.
Keyword
clean code
Vulnerability
- contracts/governance/NounsDAOInterfaces.sol#L156
- contracts/governance/NounsDAOLogicV1.sol#L507
- contracts/governance/NounsDAOLogicV2.sol#L599
일반적으로 소스코드는 80자로 제한한다. GitHub의 경우 164를 초과하는 경우 스크롤 막대를 사용해 옆으로 이동해야 한다. 따라서 줄을 분할하라.
Impact
읽기가 힘들다.
Mitigation
코드 라인을 나눈다.
Memo
뭐 이딴게… 게다가 코드도 아니고 걍 주석이 길다고…
tags: bughunting, nouns dao, clean code, smart contract, solidity, severity none