Does Lock.addressesWithdrawLocks need to be initialized?

1Hive / time-lock-app

The Time Lock app enables Aragon organizations to require users transfer or temporarily lock tokens before forwarding an intent.

GNU General Public License v3.0

16 stars 11 forks source link

Closed ghost closed 5 years ago

ghost commented 5 years ago

Lock.addressesWithdrawLocks (Lock.sol#37) is never initialized. It is used in:

forward (Lock.sol#142-158)
withdrawTokens (Lock.sol#90-93)
getWithdrawLocksCount (Lock.sol#160-162)
getGriefing (Lock.sol#170-181)
_withdrawTokens (Lock.sol#183-205)

Initialize all the variables. If a variable is meant to be initialized to zero, explicitly set it to zero.

fabriziovigevani commented 5 years ago

mappings aren't needed to be initialized.