Daniel Cousens
Daniel Cousens
This isn't a bug, or really our problem. However, I am not against removing the strict need for `messagePrefix` from our network object - as it is not needed by...
@imerkle, if you could please provide reproduction as to how you received the error in the title, that would be appreciated.
@fanatid it is probably OK, but may need the length prefix reflected to the actual length... ``` js 'Bitcoin Signed Message:\n'.length === 0x18 'Dash Signed Message:\n'.length === 0x15 ```
@mfaizan1 what is your use case for `classifyOutput`?
Please don't
@calvinmetcalf should be ready to merge straight up
LGTM, maybe @ForbesLindesay can merge it for you
@DavidLiuBlockChainTech I recommend you isolate the problem further and determine which of those functions are taking the most time, to avoid pointless speculation on our part.
@chronark can you restore this fork? I can't merge as you have deleted your fork, but I am happy to merge the changes. If you can't easily do that, I'll...
See https://github.com/feross/buffer/pull/334