- Aug 05, 2015
-
-
Jeremiah Senkpiel authored
PR-URL: https://github.com/nodejs/io.js/pull/2224 Reviewed-By:
Michaël Zasso <mic.besace@gmail.com> Reviewed-By:
Chris Dickinson <christopher.s.dickinson@gmail.com> Reviewed-By:
Roman Reiss <me@silverwind.io>
-
Jeremiah Senkpiel authored
PR-URL: https://github.com/nodejs/io.js/pull/2224 Reviewed-By:
-
Michaël Zasso authored
A bug on ARMv6 makes the process crash when the V8 debugger is used. Disabling vector ICs as a workaround. V8-Bug: https://code.google.com/p/v8/issues/detail?id=4338 PR-URL: https://github.com/nodejs/io.js/pull/2220 Reviewed-By:
Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By:
Ali Ijaz Sheikh <ofrobots@google.com> Reviewed-By:
Rod Vagg <rod@vagg.org>
-
Michaël Zasso authored
Includes cherry-picks for: * JitCodeEvent patch: https://crrev.com/f7969b1d5a55e66237221a463daf422ac7611788 * argparse patch: https://crrev.com/44bc918458481d60b08d5566f0f31a79e39b85d7 PR-URL: https://github.com/nodejs/io.js/pull/2220 Reviewed-By:
-
Brendan Ashworth authored
This commit fixes agent.getName(), which returned an extra colon according to the docs, and adds tests (it was previously not unit tested). PR-URL: https://github.com/nodejs/io.js/pull/1617 Reviewed-By:
-
Rich Trott authored
If tab completion is not being used, allow user to enter tab characters. PR-URL: https://github.com/nodejs/io.js/pull/1761 Reviewed-By:
Brendan Ashworth <brendan.ashworth@me.com>
-
Frederic Hemberger authored
PR-URL: https://github.com/nodejs/io.js/pull/2247 Reviewed-By:
-
Rod Vagg authored
Accidentally left when upgrading to V8 4.4.63.9 PR-URL: https://github.com/nodejs/io.js/pull/2275 Reviewed-By:
Jeremiah Senkpiel <fishrock123@rocketmail.com>
-
Trevor Norris authored
The new implementation of Buffer missed the check for null/undefined as the first argument to new Buffer(). Reintroduce the check and add test. Fix: e8734c0 "buffer: implement Uint8Array backed Buffer" Fix: https://github.com/nodejs/io.js/issues/2194 PR-URL: https://github.com/nodejs/io.js/pull/2195 Reviewed-By:
Colin Ihrig <cjihrig@gmail.com> Reviewed-By:
-
Rod Vagg authored
Accidentally removed @ https://github.com/nodejs/io.js/pull/2124 when ppc support was added PR-URL: https://github.com/nodejs/io.js/pull/2233 Reviewed-By:
Johan Bergstrom <bugs@bergstroem.nu> Reviewed-By:
-
Michael Dawson authored
These are the core changes that allow pLinux BE/LE compile. They don't include all of the changes needed for AIX which will follow once we have pLinux up and running in the builds PR-URL: https://github.com/nodejs/io.js/pull/2124 Reviewed-By:
Ben Noordhuis <ben@strongloop.com> Reviewed-By:
-
Trevor Norris authored
Throwing a JS error from C++ does not mean the function will return early. This must be done manually. Also remove extraneous comment no longer relevant. Fix: 2903030 "buffer: switch API to return MaybeLocal<T>" PR-URL: https://github.com/nodejs/io.js/pull/2225 Reviewed-By:
-
Rod Vagg authored
PR-URL: https://github.com/nodejs/io.js/pull/2171 Reviewed-By:
Alexis Campailla <alexis@janeasystems.com> Reviewed-By:
-
Rod Vagg authored
PR-URL: https://github.com/nodejs/io.js/pull/2154 Reviewed-By:
James M Snell <jasnell@gmail.com> Reviewed-By:
-
jomo authored
* adds missing HTTP status codes * corrects those with a wrong description * the falsely included codes have been kept PR-URL: https://github.com/nodejs/io.js/pull/1470 Reviewed-By:
Brian White <mscdex@mscdex.net> Reviewed-By:
-
Fedor Indutny authored
Put the `...^` arrow string to the hidden property of the object, and use it only when printing error to the stderr. Fix: https://github.com/nodejs/io.js/issues/2104 PR-URL: https://github.com/nodejs/io.js/pull/2108 Reviewed-By:
Trevor Norris <trev.norris@gmail.com>
-
Rod Vagg authored
PR-URL: https://github.com/nodejs/io.js/pull/2096 Reviewed-By:
-
Ben Noordhuis authored
PR-URL: https://github.com/nodejs/io.js/pull/2092 Reviewed-By:
-
Rod Vagg authored
-
Rod Vagg authored
Temporary fix only, node-gyp needs to be aware of whether it can even download this file for the current runtime so that information needs to come from the process itself. Floating patch on npm. PR-URL: https://github.com/nodejs/io.js/pull/2066 Reviewed-By:
Ryan Graham <ryan@strongloop.com>
-
Rod Vagg authored
A temporary fix only, node-gyp needs to be made more intelligent upstream about figuring out where to find the download file by inspecting the binary. Floating patch on npm. PR-URL: https://github.com/nodejs/io.js/pull/2066 Reviewed-By:
Ryan Graham <r.m.graham@gmail.com>
-
Ben Noordhuis authored
The old smalloc-based implementation has been removed, the typed array implementation is the only one that is left now. PR-URL: https://github.com/nodejs/io.js/pull/2022 Reviewed-By:
-
Ben Noordhuis authored
Remove two tests that assume creating big buffers fails. The size limit is twice as big on 64 bits architectures now and is going to be removed completely in the not too distant future. PR-URL: https://github.com/nodejs/io.js/pull/2022 Reviewed-By:
-
Ben Noordhuis authored
Fix a strict mode violation that made the test fail to run. It appears to be a regression that was introduced in commit f29762f4 ("test: enable linting for tests") which I was the sole reviewer of, so mea culpa. PR-URL: https://github.com/nodejs/io.js/pull/2022 Reviewed-By:
-
Ben Noordhuis authored
Upgrade the bundled V8 and update code in src/ and lib/ to the new API. Notable backwards incompatible changes are the removal of the smalloc module and dropped support for CESU-8 decoding. CESU-8 support can be brought back if necessary by doing UTF-8 decoding ourselves. This commit includes https://codereview.chromium.org/1192973004 to fix a build error on python 2.6 systems. The original commit log follows: Use optparse in js2c.py for python compatibility Without this change, V8 won't build on RHEL/CentOS 6 because the distro python is too old to know about the argparse module. PR-URL: https://github.com/nodejs/io.js/pull/2022 Reviewed-By:
-
Trevor Norris authored
Buffer now uses the ArrayBuffer as the backing store if passed to the constructor. Fixes: https://github.com/nodejs/io.js/issues/106 PR-URL: https://github.com/nodejs/io.js/pull/2002 Reviewed-By:
Domenic Denicola <d@domenic.me>
-
Trevor Norris authored
PR-URL: https://github.com/nodejs/io.js/pull/2003 Reviewed-By:
-
Trevor Norris authored
Starting in V8 v4.3 the maximum array index of a typed array is the same as the largest Smi supported on a given architecture. To compensate for these differences export kMaxLength from the buffer module with the correct size for the given architecture. PR-URL: https://github.com/nodejs/io.js/pull/2003 Reviewed-By:
-
cjihrig authored
Currently, each cluster worker is assigned an ever increasing --debug-port argument. A long running cluster application that does not use the debugger can run into errors related to the port range. This commit mitigates the problem by only setting the debug port if the master is started with debug arguments, or the user explicitly defines debug arguments for the worker. This commit also adds a new debug port offset counter that is only incremented when a worker is created that utilizes debugging. Fixes: https://github.com/joyent/node/issues/8159 Refs: https://github.com/nodejs/io.js/pull/1524 PR-URL: https://github.com/nodejs/io.js/pull/1949 Reviewed-By:
Oleg Elifantiev <oleg@elifantiev.ru>
-
Sakthipriyan Vairamani authored
Fix Buffer.concat() so a copy is always returned regardless of the number of buffers that were passed. Previously if the array length was one then the same same buffer was returned. This created a special case for the user where there was a chance mutating the buffer returned by .concat() could mutate the buffer passed in. Also fixes an inconsistency when throwing if an array member was not a Buffer instance. For example: Buffer.concat([42]); // Returns 42 Buffer.concat([42, 1]); // Throws a TypeError Now .concat() will always throw if an array member is not a Buffer instance. See: https://github.com/nodejs/io.js/issues/1891 PR-URL: https://github.com/nodejs/io.js/pull/1937 Reviewed-By: -
Fedor Indutny authored
Do not detach the socket from the response until all data is actually sent to the other side. See: https://github.com/iojs/io.js/pull/1373 PR-URL: https://github.com/nodejs/io.js/pull/1411 Reviewed-By:
-
Matteo Collina authored
Modifies the dgram send() method to not emit errors when a DNS lookup fails if there is a callback. Given that the same UDP socket can be used to send messages to different hosts, the socket can be reused even if one of those send() fails. This slightly changes the behavior of a stable API, so that it behaves as users would expect to. This is based on https://github.com/joyent/node/pull/7738, which landed in 77266d7fadd8dfefb107ccb1e3fe97f9620f1288. Fixes: https://github.com/joyent/node/issues/4846 Refs: https://github.com/joyent/node/pull/7738 PR-URL: https://github.com/nodejs/io.js/pull/1796 Reviewed-By:
-
Chris Dickinson authored
This allows users to provide a callback that handles potential errors resulting from a `socket.send` call. The original behavior of emitting the error event on the socket is preserved. Fixes: https://github.com/joyent/node/issues/4846 PR-URL: https://github.com/joyent/node/pull/7738 PR-URL: https://github.com/nodejs/io.js/pull/1796 Reviewed-By:
-
Domenic Denicola authored
By using the new SetHandler API instead of SetNamedPropertyHandler, we can intercept symbols now. This forces us to use Maybes and MaybeLocals more, since this new API does not have a non-maybe variant. Fixes: https://github.com/nodejs/io.js/issues/884 PR-URL: https://github.com/nodejs/io.js/pull/1773 Reviewed-By:
Fedor Indutny <fedor@indutny.com> Reviewed-By:
-
Domenic Denicola authored
No reason to install access checks if they're always going to return true. PR-URL: https://github.com/nodejs/io.js/pull/1773 Reviewed-By:
-
Domenic Denicola authored
The GlobalPropertyQueryCallback was changed in 2010 to return an integer instead of a boolean: https://groups.google.com/forum/#!topic/v8-users/OOjHJrix-cU This integer communicates the property descriptors of the property, instead of just its presence or absence. However, the original contextify code was probably written before this change, and it was not updated when porting to Node.js. Credit to @smikes for the test and the original PR of #885. Fixes: https://github.com/nodejs/io.js/pull/864 Fixes: https://github.com/nodejs/io.js/pull/885 PR-URL: https://github.com/nodejs/io.js/pull/1773 Reviewed-By:
-
Trevor Norris authored
New Buffer implementation allows greater than kMaxLength to be created. So instead check if the passed value is a valid Smi. PR-URL: https://github.com/nodejs/io.js/pull/1825 Reviewed-By:
-
Trevor Norris authored
Address comments and deprecations left in source files. These changes include: * Remove the deprecated API. * Change Buffer::New() that did a copy of the data to Buffer::Copy() * Change Buffer::Use() to Buffer::New() PR-URL: https://github.com/nodejs/io.js/pull/1825 Reviewed-By:
-
Trevor Norris authored
Instead of aborting in case of internal failure, return an empty Local<Object>. Using the MaybeLocal<T> API, users must check their return values. PR-URL: https://github.com/nodejs/io.js/pull/1825 Reviewed-By:
-
Trevor Norris authored
Use the new Maybe<T> syntax for v8::Object::SetPrototype(). PR-URL: https://github.com/nodejs/io.js/pull/1825 Reviewed-By:
-
