src,test: drop SlowBuffer

benchmark/buffers/buffer-iterate.js

@@ -1,11 +1,9 @@
 'use strict';
-const SlowBuffer = require('buffer').SlowBuffer;
 const common = require('../common.js');
 const assert = require('assert');
 
 const bench = common.createBenchmark(main, {
   size: [512, 4096, 16386],
-  type: ['fast'],
   method: ['for', 'forOf', 'iterator'],
   n: [1e3],
 });
@@ -16,10 +14,8 @@ const methods = {
   'iterator': benchIterator,
 };
 
-function main({ size, type, method, n }) {
-  const buffer = type === 'fast' ?
-    Buffer.alloc(size) :
-    SlowBuffer(size).fill(0);
+function main({ size, method, n }) {
+  const buffer = Buffer.alloc(size);
 
   const fn = methods[method];
 

benchmark/buffers/buffer-read-with-byteLength.js

@@ -9,16 +9,13 @@ const types = [
 ];
 
 const bench = common.createBenchmark(main, {
-  buffer: ['fast'],
   type: types,
   n: [1e6],
   byteLength: [1, 2, 3, 4, 5, 6],
 });
 
-function main({ n, buf, type, byteLength }) {
-  const buff = buf === 'fast' ?
-    Buffer.alloc(8) :
-    require('buffer').SlowBuffer(8);
+function main({ n, type, byteLength }) {
+  const buff = Buffer.alloc(8);
   const fn = `read${type}`;
 
   buff.writeDoubleLE(0, 0);

benchmark/buffers/buffer-read.js

@@ -19,15 +19,12 @@ const types = [
 ];
 
 const bench = common.createBenchmark(main, {
-  buffer: ['fast'],
   type: types,
   n: [1e6],
 });
 
-function main({ n, buf, type }) {
-  const buff = buf === 'fast' ?
-    Buffer.alloc(8) :
-    require('buffer').SlowBuffer(8);
+function main({ n, type }) {
+  const buff = Buffer.alloc(8);
   const fn = `read${type}`;
 
   buff.writeDoubleLE(0, 0);

benchmark/buffers/buffer-slice.js

@@ -1,17 +1,15 @@
 'use strict';
 const common = require('../common.js');
-const SlowBuffer = require('buffer').SlowBuffer;
 
 const bench = common.createBenchmark(main, {
-  type: ['fast', 'slow', 'subarray'],
+  type: ['slice', 'subarray'],
   n: [1e6],
 });
 
 const buf = Buffer.allocUnsafe(1024);
-const slowBuf = new SlowBuffer(1024);
 
 function main({ n, type }) {
-  const b = type === 'slow' ? slowBuf : buf;
+  const b = buf
   const fn = type === 'subarray' ?
     () => b.subarray(10, 256) :
     () => b.slice(10, 256);

benchmark/buffers/buffer-write.js

@@ -27,7 +27,6 @@ const types = [
 ];
 
 const bench = common.createBenchmark(main, {
-  buffer: ['fast'],
   type: types,
   n: [1e6],
 });
@@ -70,10 +69,8 @@ const byteLength = {
   writeIntBE: 6,
 };
 
-function main({ n, buf, type }) {
-  const buff = buf === 'fast' ?
-    Buffer.alloc(8) :
-    require('buffer').SlowBuffer(8);
+function main({ n, type }) {
+  const buff = Buffer.alloc(8)
   const fn = `write${type}`;
 
   if (!/\d/.test(fn))

doc/api/buffer.md

@@ -5326,30 +5326,6 @@ console.log(newBuf.toString('ascii'));
 Because the Euro (`€`) sign is not representable in US-ASCII, it is replaced
 with `?` in the transcoded `Buffer`.
 
-### Class: `SlowBuffer`
-
-<!-- YAML
-deprecated: v6.0.0
--->
-
-> Stability: 0 - Deprecated: Use [`Buffer.allocUnsafeSlow()`][] instead.
-
-See [`Buffer.allocUnsafeSlow()`][]. This was never a class in the sense that
-the constructor always returned a `Buffer` instance, rather than a `SlowBuffer`
-instance.
-
-#### `new SlowBuffer(size)`
-
-<!-- YAML
-deprecated: v6.0.0
--->
-
-> Stability: 0 - Deprecated: Use [`Buffer.allocUnsafeSlow()`][] instead.
-
-* `size` {integer} The desired length of the new `SlowBuffer`.
-
-See [`Buffer.allocUnsafeSlow()`][].
-
 ### Buffer constants
 
 <!-- YAML
@@ -5483,7 +5459,7 @@ added: v5.10.0
 Node.js can be started using the `--zero-fill-buffers` command-line option to
 cause all newly-allocated `Buffer` instances to be zero-filled upon creation by
 default. Without the option, buffers created with [`Buffer.allocUnsafe()`][],
-[`Buffer.allocUnsafeSlow()`][], and `new SlowBuffer(size)` are not zero-filled.
+and [`Buffer.allocUnsafeSlow()`][] are not zero-filled.
 Use of this flag can have a measurable negative impact on performance. Use the
 `--zero-fill-buffers` option only when necessary to enforce that newly allocated
 `Buffer` instances cannot contain old data that is potentially sensitive.

doc/api/cli.md

@@ -2833,8 +2833,7 @@ node --watch --watch-preserve-output test.js
 added: v6.0.0
 -->
 
-Automatically zero-fills all newly allocated [`Buffer`][] and [`SlowBuffer`][]
-instances.
+Automatically zero-fills all newly allocated [`Buffer`][] instances.
 
 ## Environment variables
 
@@ -3580,7 +3579,6 @@ node --stack-trace-limit=12 -p -e "Error.stackTraceLimit" # prints 12
 [`CRYPTO_secure_malloc_init`]: https://www.openssl.org/docs/man3.0/man3/CRYPTO_secure_malloc_init.html
 [`NODE_OPTIONS`]: #node_optionsoptions
 [`NO_COLOR`]: https://no-color.org
-[`SlowBuffer`]: buffer.md#class-slowbuffer
 [`Web Storage`]: https://developer.mozilla.org/en-US/docs/Web/API/Web_Storage_API
 [`WebSocket`]: https://developer.mozilla.org/en-US/docs/Web/API/WebSocket
 [`YoungGenerationSizeFromSemiSpaceSize`]: https://chromium.googlesource.com/v8/v8.git/+/refs/tags/10.3.129/src/heap/heap.cc#328

doc/api/deprecations.md

@@ -691,23 +691,6 @@ Type: End-of-Life
 
 `util.error()` has been removed. Please use [`console.error()`][] instead.
 
-### DEP0030: `SlowBuffer`
-
-<!-- YAML
-changes:
-  - version: v6.12.0
-    pr-url: https://github.com/nodejs/node/pull/10116
-    description: A deprecation code has been assigned.
-  - version: v6.0.0
-    pr-url: https://github.com/nodejs/node/pull/5833
-    description: Documentation-only deprecation.
--->
-
-Type: Documentation-only
-
-The [`SlowBuffer`][] class is deprecated. Please use
-[`Buffer.allocUnsafeSlow(size)`][] instead.
-
 ### DEP0031: `ecdh.setPublicKey()`
 
 <!-- YAML
@@ -3761,7 +3744,6 @@ It is recommended to use the `new` qualifier instead. This applies to all REPL c
 [`ReadStream.open()`]: fs.md#class-fsreadstream
 [`Server.getConnections()`]: net.md#servergetconnectionscallback
 [`Server.listen({fd: <number>})`]: net.md#serverlistenhandle-backlog-callback
-[`SlowBuffer`]: buffer.md#class-slowbuffer
 [`String.prototype.toWellFormed`]: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/toWellFormed
 [`WriteStream.open()`]: fs.md#class-fswritestream
 [`assert.CallTracker`]: assert.md#class-assertcalltracker

doc/node.1

@@ -609,7 +609,7 @@ If set to 0 then V8 will choose an appropriate size of the thread pool based on
 If the value provided is larger than V8's maximum, then the largest value will be chosen.
 .
 .It Fl -zero-fill-buffers
-Automatically zero-fills all newly allocated Buffer and SlowBuffer instances.
+Automatically zero-fills all newly allocated Buffer instances.
 .
 .It Fl c , Fl -check
 Check the script's syntax without executing it.

lib/buffer.js

@@ -407,7 +407,7 @@ Buffer.allocUnsafe = function allocUnsafe(size) {
 };
 
 /**
- * Equivalent to SlowBuffer(num), by default creates a non-zero-filled
+ * By default creates a non-zero-filled
  * Buffer instance that is not allocated off the pre-initialized pool.
  * If `--zero-fill-buffers` is set, will zero-fill the buffer.
  */
@@ -416,16 +416,6 @@ Buffer.allocUnsafeSlow = function allocUnsafeSlow(size) {
   return createUnsafeBuffer(size);
 };
 
-// If --zero-fill-buffers command line argument is set, a zero-filled
-// buffer is returned.
-function SlowBuffer(size) {
-  validateNumber(size, 'size', 0, kMaxLength);
-  return createUnsafeBuffer(size);
-}
-
-ObjectSetPrototypeOf(SlowBuffer.prototype, Uint8ArrayPrototype);
-ObjectSetPrototypeOf(SlowBuffer, Uint8Array);
-
 function allocate(size) {
   if (size <= 0) {
     return new FastBuffer();
@@ -1322,7 +1312,6 @@ function isAscii(input) {
 
 module.exports = {
   Buffer,
-  SlowBuffer,
   transcode,
   isUtf8,
   isAscii,

src/node_options.cc

@@ -978,8 +978,7 @@ PerProcessOptionsParser::PerProcessOptionsParser(
             &PerProcessOptions::v8_thread_pool_size,
             kAllowedInEnvvar);
   AddOption("--zero-fill-buffers",
-            "automatically zero-fill all newly allocated Buffer and "
-            "SlowBuffer instances",
+            "automatically zero-fill all newly allocated Buffer",
             &PerProcessOptions::zero_fill_all_buffers,
             kAllowedInEnvvar);
   AddOption("--debug-arraybuffer-allocations",

test/parallel/test-buffer-alloc.js

@@ -5,7 +5,6 @@ const assert = require('assert');
 const vm = require('vm');
 
 const {
-  SlowBuffer,
   kMaxLength,
 } = require('buffer');
 
@@ -1104,8 +1103,6 @@ assert.throws(() => Buffer.from(null), {
 // Test prototype getters don't throw
 assert.strictEqual(Buffer.prototype.parent, undefined);
 assert.strictEqual(Buffer.prototype.offset, undefined);
-assert.strictEqual(SlowBuffer.prototype.parent, undefined);
-assert.strictEqual(SlowBuffer.prototype.offset, undefined);
 
 
 {

test/parallel/test-buffer-bytelength.js

@@ -2,7 +2,6 @@
 
 const common = require('../common');
 const assert = require('assert');
-const SlowBuffer = require('buffer').SlowBuffer;
 const vm = require('vm');
 
 [
@@ -24,7 +23,6 @@ const vm = require('vm');
 });
 
 assert(ArrayBuffer.isView(new Buffer(10)));
-assert(ArrayBuffer.isView(new SlowBuffer(10)));
 assert(ArrayBuffer.isView(Buffer.alloc(10)));
 assert(ArrayBuffer.isView(Buffer.allocUnsafe(10)));
 assert(ArrayBuffer.isView(Buffer.allocUnsafeSlow(10)));

test/parallel/test-buffer-failed-alloc-typed-arrays.js

@@ -2,7 +2,6 @@
 
 require('../common');
 const assert = require('assert');
-const SlowBuffer = require('buffer').SlowBuffer;
 
 // Test failed or zero-sized Buffer allocations not affecting typed arrays.
 // This test exists because of a regression that occurred. Because Buffer
@@ -15,7 +14,6 @@ const zeroArray = new Uint32Array(10).fill(0);
 const sizes = [1e20, 0, 0.1, -1, 'a', undefined, null, NaN];
 const allocators = [
   Buffer,
-  SlowBuffer,
   Buffer.alloc,
   Buffer.allocUnsafe,
   Buffer.allocUnsafeSlow,

test/parallel/test-buffer-inspect.js

@@ -30,29 +30,20 @@ buffer.INSPECT_MAX_BYTES = 2;
 let b = Buffer.allocUnsafe(4);
 b.fill('1234');
 
-let s = buffer.SlowBuffer(4);
-s.fill('1234');
-
 let expected = '<Buffer 31 32 ... 2 more bytes>';
 
 assert.strictEqual(util.inspect(b), expected);
-assert.strictEqual(util.inspect(s), expected);
 
 b = Buffer.allocUnsafe(2);
 b.fill('12');
 
-s = buffer.SlowBuffer(2);
-s.fill('12');
-
 expected = '<Buffer 31 32>';
 
 assert.strictEqual(util.inspect(b), expected);
-assert.strictEqual(util.inspect(s), expected);
 
 buffer.INSPECT_MAX_BYTES = Infinity;
 
 assert.strictEqual(util.inspect(b), expected);
-assert.strictEqual(util.inspect(s), expected);
 
 b.inspect = undefined;
 b.prop = new Uint8Array(0);

test/parallel/test-buffer-no-negative-allocation.js

@@ -2,7 +2,6 @@
 
 require('../common');
 const assert = require('assert');
-const { SlowBuffer } = require('buffer');
 
 const msg = {
   code: 'ERR_OUT_OF_RANGE',
@@ -30,8 +29,3 @@ assert.throws(() => Buffer.allocUnsafeSlow(-Buffer.poolSize), msg);
 assert.throws(() => Buffer.allocUnsafeSlow(-100), msg);
 assert.throws(() => Buffer.allocUnsafeSlow(-1), msg);
 assert.throws(() => Buffer.allocUnsafeSlow(NaN), msg);
-
-assert.throws(() => SlowBuffer(-Buffer.poolSize), msg);
-assert.throws(() => SlowBuffer(-100), msg);
-assert.throws(() => SlowBuffer(-1), msg);
-assert.throws(() => SlowBuffer(NaN), msg);

test/parallel/test-buffer-over-max-length.js

@@ -4,7 +4,6 @@ require('../common');
 const assert = require('assert');
 
 const buffer = require('buffer');
-const SlowBuffer = buffer.SlowBuffer;
 
 const kMaxLength = buffer.kMaxLength;
 const bufferMaxSizeMsg = {
@@ -13,7 +12,6 @@ const bufferMaxSizeMsg = {
 };
 
 assert.throws(() => Buffer(kMaxLength + 1), bufferMaxSizeMsg);
-assert.throws(() => SlowBuffer(kMaxLength + 1), bufferMaxSizeMsg);
 assert.throws(() => Buffer.alloc(kMaxLength + 1), bufferMaxSizeMsg);
 assert.throws(() => Buffer.allocUnsafe(kMaxLength + 1), bufferMaxSizeMsg);
 assert.throws(() => Buffer.allocUnsafeSlow(kMaxLength + 1), bufferMaxSizeMsg);