search

tokamak-network / Tms-contract

GNU General Public License v3.0
0 stars 1 forks source link

Remove the zero assignments #11

Closed usgeeus closed 2 months ago

usgeeus commented 2 months ago

Description Just declare the variable and remove the 0 assignment. The default value of the variable is 0 anyway.

The one of purpose of the multisend is to save gas.

Lines of Codes https://github.com/tokamak-network/Tms-contract/blob/31a33830d7840ef742a24d1be3b299d6a1a05c07/contracts/MultiSender.sol#L46 https://github.com/tokamak-network/Tms-contract/blob/31a33830d7840ef742a24d1be3b299d6a1a05c07/contracts/MultiSender.sol#L94

Configuration

Impact Waste gas.

Recommendation

uint256 _failedAmount; uint256 _totalAmount;

usgeeus commented 2 months ago

Solved