-
-
Notifications
You must be signed in to change notification settings - Fork 230
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Consider matching Buffer.toJSON() more closely #34
Comments
Actually, this is how buffer.toJSON() works in node 0.11 which this module 0.11 also made some other changes including adding buffer.equal(), On Thursday, July 10, 2014, David Humphrey notifications@github.com wrote:
Feross | blog http://feross.org/ | webtorrent http://webtorrent.io/ | |
Ah, that's really useful, thanks for the info. You should consider calling this out in your README. |
Added a note to the readme, thanks! |
With
buffer
in the browser I get{type: 'Buffer', data: [...]}
. Obviously you're doing this intentionally, since the code does exactly this. However, I wonder why you've chosen to do it this way? At the very least, documenting the difference will save users problems when it works differently than they expect.The text was updated successfully, but these errors were encountered: