lib: remove internal `util._extends()` usage

This removes all internal calls to the deprecated `_extends()`
function. It is slower than `Object.assign()` and the object spread
notation since V8 6.8 and using the spread notation often also
results in shorter code.

PR-URL: https://github.com/nodejs/node/pull/25105
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>

1 files changed, 5 insertions, 6 deletions

diff --git a/lib/tty.js b/lib/tty.js
index 62945a78eb..6efaac6e43 100644
--- a/lib/tty.js
+++ b/lib/tty.js
@@ -21,7 +21,7 @@
 
 'use strict';
 
-const { inherits, _extend } = require('util');
+const { inherits } = require('util');
 const net = require('net');
 const { TTY, isTTY } = internalBinding('tty_wrap');
 const errors = require('internal/errors');
@@ -47,14 +47,13 @@ function ReadStream(fd, options) {
     throw new ERR_TTY_INIT_FAILED(ctx);
   }
 
-  options = _extend({
+  net.Socket.call(this, {
     highWaterMark: 0,
     readable: true,
     writable: false,
-    handle: tty
-  }, options);
-
-  net.Socket.call(this, options);
+    handle: tty,
+    ...options
+  });
 
   this.isRaw = false;
   this.isTTY = true;