search

Siorki / RegPack

Self-contained packer for size-constrained JS code
Other
301 stars 14 forks source link

Unable to allocate a variable name for setInterval() #88

Closed Siorki closed 6 years ago

Siorki commented 6 years ago

Option "refactor with setInterval()" crashes RegPack if the variable name is not provided. Regression spotted in v5.0.2

Siorki commented 6 years ago

allocateNewVariable() does not get the correct parameters.

Suggested fix : use a PackerData and reach for the contents instead.

Siorki commented 6 years ago

Changed signature for allocateNewVariable()

https://github.com/Siorki/RegPack/commit/9efcc14832adcea9f93d87f5f11707c5a21bc2bd