regression in node 8
See original GitHub issueadded node 8 to node-mysql2 build and one of cesu8 tests fails now. Simple example to reproduce:
node 0.10.48 ( any pre 8.x )
> var iconv = require('iconv-lite')
> iconv.decode(Buffer('eda0bdedb2a9', 'hex'), 'cesu8')
'💩'
> Buffer(iconv.decode(Buffer('eda0bdedb2a9', 'hex'), 'cesu8'))
<Buffer f0 9f 92 a9>
node 8.1.0:
> var iconv = require('iconv-lite')
undefined
> iconv.decode(Buffer('eda0bdedb2a9', 'hex'), 'cesu8')
'��'
> Buffer.from(iconv.decode(Buffer.from('eda0bdedb2a9', 'hex'), 'cesu8'))
<Buffer ef bf bd ef bf bd>
Issue Analytics
- State:
- Created 6 years ago
- Comments:10 (5 by maintainers)
Top Results From Across the Web
Major performance regression from 6.10.2 to 7.x and 8.0.0-test
I'm seeing a major performance regression from Node 6.x (e.g. 6.10.2) to all versions of Node 7.x and v8.0.0-test20170511830c4bf319.
Read more >Linear & Non Linear Regression In Node Js - YouTube
A quick walkthrough of statistical regression analysis with the npm ... Linear & Non Linear Regression In Node Js ... 3 views 8...
Read more >regression - npm
regression -js is a JavaScript module containing a collection of linear least-squares fitting methods for simple data analysis. Installation.
Read more >Regression Node - SAS Help Center
The Regression node supports forward, backward, and stepwise selection methods. Data sets that have a role of score are automatically scored ...
Read more >Regression Predictor - NodePit
Predicts the response using a regression model. The node needs to be connected to a regression node model* and some test data. It...
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
@chalker I had 3 bug reports next day after I accidentally released node-mysql2 with one of the functions using new API without shim, there are still plenty of pre node 4.5 users
Thanks for the report! I reproduced it and will fix soon.
I’ll need to replace the check here: https://github.com/ashtuchkin/iconv-lite/blob/master/encodings/internal.js#L38 to something like
new Buffer('eda0bdedb2a9', 'hex').toString() !== '💩'