-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
Add node/string_decoder and some more #6638
Conversation
I might need some orientation on how to convert a function constructor to a class data type. This was terribly designed, since it expects to get an object with properties that may or may not exist depending on the arguments passed. I don't even think it's a valid argument to say that because JS is so flexible this kind of stuff is valid, this is just plain bad code.
Edit: Fixed through the ugliest of ways, but it will do for now. If you have any suggestions about this please let me know |
Add tests for supported encodings
@marcosc90 could you review this PR? |
Yes, I'll review it properly over the weekend @Soremwar |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - thank you much @Soremwar
ucs2To be added in next PRutf16leTo be added in next PRlatin1To be added in next PRbinaryTo be added in next PRasciiTo be added in next PRNote
Should we keep the copyright comments?