[internal test - Contract Bugs]:

[cenwadike]

Describe the bug

Not bugs per se. Suggestions for improving code quality.

• /contracts/layer2/L1BridgeRegistryV1_1.sol:
  • line 159 - line 179 Remove unused commented code.
  • _availableForRegistration: remove unused variable _name
  • line 411 - line 413: Remove unused commented conditional.
• /contracts/layer2/Layer2ManagerV1_1.sol:
  • contract Layer2ManagerV1_1 -> contract Layer2ManagerV1_1
• /contracts/layer2/OperatorManagerProxy.sol
  • remove excess empty line at the end of the contract
• /contracts/stake/managers/SeigManagerV1_3.sol:
  • keep code within 100 Columns to improve readability
• /contracts/stake/managers/DepositManagerV1_1.sol:
  • keep code within 100 Columns to improve readability
• /contracts/dao/DAOCommittee_V1.sol:
  • rename ADDMEMBER_FAIL to ADD_MEMBER_FAIL
• /contracts/proxy/DAOCommitteeProxy2.sol:
  • keep code within 100 Columns to improve readability

Impact

Suggestions aim to improve the coding standard and improve readability and consistent coding standard

Exploit Scenario

NA

Recommendation

Recommendations are found in the description

[Zena-park]

/contracts/layer2/L1BridgeRegistryV1_1.sol:
line 159 - line 179 Remove unused commented code.
_availableForRegistration: remove unused variable _name
line 411 - line 413: Remove unused commented conditional.

It has already been fixed. https://github.com/tokamak-network/ton-staking-v2/commit/37bac22659d03ac9e50455da3412ccf3ab5a6810

[Zena-park]

@zzooppii Please check related to DAOCommittee_V1.sol

[Zena-park]

@cenwadike Everything mentioned in the issue has been reflected.