readline: use kEmptyObject

PR-URL: #43159 Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
nodejs · Jul 31, 2022 · 37066f0 · 37066f0
1 parent f08a282
commit 37066f0
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/lib/internal/readline/interface.js b/lib/internal/readline/interface.js
@@ -43,6 +43,7 @@ const {
   validateString,
   validateUint32,
 } = require('internal/validators');
+const { kEmptyObject } = require('internal/util');
 const {
   inspect,
   getStringWidth,
@@ -922,7 +923,7 @@ class Interface extends InterfaceConstructor {
   // Handle a write from the tty
   [kTtyWrite](s, key) {
     const previousKey = this[kPreviousKey];
-    key = key || {};
+    key = key || kEmptyObject;
     this[kPreviousKey] = key;
 
     // Activate or deactivate substring search.

diff --git a/lib/readline.js b/lib/readline.js
@@ -100,7 +100,10 @@ const {
   kSubstringSearch,
 } = require('internal/readline/utils');
 
-const { promisify } = require('internal/util');
+const {
+  kEmptyObject,
+  promisify,
+} = require('internal/util');
 
 const { StringDecoder } = require('string_decoder');
 
@@ -987,7 +990,7 @@ Interface.prototype._moveCursor = function(dx) {
 };
 
 function _ttyWriteDumb(s, key) {
-  key = key || {};
+  key = key || kEmptyObject;
 
   if (key.name === 'escape') return;