Skip to content

Commit 6a0f463

Browse files
committed
buffer: avoid materializing ArrayBuffer for creation
Do not create an `ArrayBuffer` if the engine’s settings avoid it and we don’t need it. PR-URL: #26301 Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: James M Snell <[email protected]>
1 parent 31975bb commit 6a0f463

File tree

1 file changed

+2
-6
lines changed

1 file changed

+2
-6
lines changed

lib/buffer.js

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -105,21 +105,17 @@ let poolSize, poolOffset, allocPool;
105105
const zeroFill = bindingZeroFill || [0];
106106

107107
function createUnsafeBuffer(size) {
108-
return new FastBuffer(createUnsafeArrayBuffer(size));
109-
}
110-
111-
function createUnsafeArrayBuffer(size) {
112108
zeroFill[0] = 0;
113109
try {
114-
return new ArrayBuffer(size);
110+
return new FastBuffer(size);
115111
} finally {
116112
zeroFill[0] = 1;
117113
}
118114
}
119115

120116
function createPool() {
121117
poolSize = Buffer.poolSize;
122-
allocPool = createUnsafeArrayBuffer(poolSize);
118+
allocPool = createUnsafeBuffer(poolSize).buffer;
123119
poolOffset = 0;
124120
}
125121
createPool();

0 commit comments

Comments
 (0)