search

bitcoinjs / indexd

An external bitcoind index management service module
ISC License
53 stars 23 forks source link

Remove extra setTimeout in Blockchain#connect #31

Closed fanatid closed 6 years ago

fanatid commented 6 years ago

Hello const/for..of/spread! 😆

dcousens commented 6 years ago

LGTM, but what was the rationale? I was going to say it changes the memory requirements, but I suppose they are the same.

fanatid commented 6 years ago

I think memory requirements will be same, because instead array you store arguments in v8 queue (or somewhere in node). IDC about PR, feel free drop/change/etc. It was first thing which I wanted to change when looked on code.