-
Notifications
You must be signed in to change notification settings - Fork 30.2k
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
test-buffer-indexof.js failing on pLinux BE #3283
Comments
@mhdawson Is it possible to get access to a big endian machine? string_search.h is mostly copied from v8 with some type changes and is tested on BE so most likely the problem is in IndexOfString in node_buffer.cc. |
yes I can get you temporary access on the BE machine if you can send me your ssh public key I sent the connection info to the email for your github account, once I get your ssh key I'll add and you'll be able to log in. |
So far I've only been able to investigate so far, but from what I can see, I believe the issue does lie in the node_buffer.cc IndexOfString changes that were introduced by #2539. I could be off but It appears that the input string argument on the buffer.indexOf is not being properly ordered properly on BE platforms so when the search is performed, it's not finding a match. |
On Big Endian platforms v8 strings are need to converted to Little Endian before searching in utf16le buffer Fixes: #3283 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Trevor Norris <[email protected]> Reviewed-By: Michael Dawson <[email protected]> PR-URL: #3295
On Big Endian platforms v8 strings are need to converted to Little Endian before searching in utf16le buffer Fixes: nodejs#3283 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Trevor Norris <[email protected]> Reviewed-By: Michael Dawson <[email protected]> PR-URL: nodejs#3295
https://ci.nodejs.org/job/iojs+pr+ppc/nodes=ppcbe-fedora20/25/console
Looks like test was added or modified by this PR #2539
@skomski can you take a look ?
@jasnell FYI
The text was updated successfully, but these errors were encountered: