BridgeAR · GitHub

@BridgeAR added buffer

Issues and PRs related to the buffer subsystem.

c++

Issues and PRs that require attention from people who are familiar with C++.

benchmark

Issues and PRs related to the benchmark subsystem.

performance

Issues and PRs related to the performance of Node.js.

labels

Jan 26, 2018

@BridgeAR

@nodejs-github-bot nodejs-github-bot added buffer

Issues and PRs related to the buffer subsystem.

c++

Issues and PRs that require attention from people who are familiar with C++.

labels

Jan 26, 2018

@BridgeAR BridgeAR added semver-major

PRs that contain breaking changes and should be released in the next major version.

wip

Issues and PRs that are still a work in progress.

labels

Jan 26, 2018

BridgeAR

@maclover7

@cjihrig

mcollina

Closed

apapirovski

This was referenced

Mar 12, 2018

Closed

Merged

Merged

kriskowal pushed a commit to thriftrw/thriftrw-node that referenced this pull request

Mar 26, 2018
Support for the `noAssert` argument dropped in the upcoming Node.js
v.10. This removes the argument to make sure everything works as it
should.
Refs: nodejs/node#18395

BridgeAR added a commit to BridgeAR/tedious that referenced this pull request

Apr 1, 2018
Support for the `noAssert` argument dropped in the upcoming Node.js
v.10. This removes the argument to make sure everything works as it
should.
Refs: nodejs/node#18395

BridgeAR added a commit to BridgeAR/thriftrw-node that referenced this pull request

Apr 1, 2018
Support for the `noAssert` argument dropped in the upcoming Node.js
v.10. This removes the argument to make sure everything works as it
should.
Refs: nodejs/node#18395

Closed

4 tasks

alexpenev-s pushed a commit to SAP/node-hdb that referenced this pull request

Apr 3, 2018
Support for the `noAssert` argument dropped in the upcoming Node.js
v.10. This removes the argument to make sure everything works as it
should.
Refs: nodejs/node#18395

BridgeAR added a commit to BridgeAR/node that referenced this pull request

Apr 13, 2018
If none is provided, use zero as a default offset for all read/write
operations on the buffer.
PR-URL: nodejs#19749
Refs: nodejs#18395
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>

jasnell pushed a commit that referenced this pull request

Apr 16, 2018
If none is provided, use zero as a default offset for all read/write
operations on the buffer.
PR-URL: #19749
Refs: #18395
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>

MayaLekova pushed a commit to MayaLekova/node that referenced this pull request

May 8, 2018
The FIXME is obsolete as it was meant about a indentation issue
that got fixed a long time ago.
PR-URL: nodejs#18395
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

MayaLekova pushed a commit to MayaLekova/node that referenced this pull request

May 8, 2018
This ports the Buffer#write(Double|Float)(B|L)E functions to JS.
This fixes a security issue concerning type confusion and fixes
another possible crash in combination with `noAssert`.
In addition to that it will also significantly improve the write
performance.
Fixes: nodejs#12179
Fixes: nodejs#8724
PR-URL: nodejs#18395
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

MayaLekova pushed a commit to MayaLekova/node that referenced this pull request

May 8, 2018
PR-URL: nodejs#18395
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

MayaLekova pushed a commit to MayaLekova/node that referenced this pull request

May 8, 2018
There are a lot of changes in this commit:
1) Remove the `noAssert` argument from all read and write functions.
2) Improve the performance of all read floating point functions
   significantly. This is done by switching to TypedArrays as the
   write floating point write functions.
3) No implicit type coercion for offset and byteLength anymore.
4) Adds a lot of tests.
5) Moves the read and write functions to the internal buffer file
   to split the files in smaller chunks.
6) Reworked a lot of existing tests.
7) Improve the performane of all all read write functions by using
   a faster input validation and by improving function logic.
8) Significantly improved the performance of all read int functions.
   This is done by using a implementation without a loop.
9) Improved error handling.
10) Rename test file to use the correct subsystem.
PR-URL: nodejs#18395
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

MayaLekova pushed a commit to MayaLekova/node that referenced this pull request

May 8, 2018
This removes the `noAssert` argument and also adds some more tests.
PR-URL: nodejs#18395
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

MayaLekova pushed a commit to MayaLekova/node that referenced this pull request

May 8, 2018
PR-URL: nodejs#18395
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

Closed

Open

fanatid

Closed

@ankon ankon mentioned this pull request

Jun 21, 2020

Merged

Open

Read the original on github.com ↗