From 88b9572d76aaef779308c2a32b994d3d6d6b7b0d Mon Sep 17 00:00:00 2001 From: Bryan English Date: Wed, 27 Sep 2017 23:18:41 -0700 Subject: [PATCH] tty: require readline at top of file No need to require it on each of those function calls. Backport-PR-URL: https://github.com/nodejs/node/pull/16264 PR-URL: https://github.com/nodejs/node/pull/15647 Reviewed-By: Ruben Bridgewater Reviewed-By: James M Snell Reviewed-By: Colin Ihrig Reviewed-By: Luigi Pinca Reviewed-By: Jeremiah Senkpiel --- lib/tty.js | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/lib/tty.js b/lib/tty.js index e836beb900f155..08977115c6e60d 100644 --- a/lib/tty.js +++ b/lib/tty.js @@ -6,6 +6,7 @@ const TTY = process.binding('tty_wrap').TTY; const isTTY = process.binding('tty_wrap').isTTY; const inherits = util.inherits; const errnoException = util._errnoException; +const readline = require('readline'); exports.isatty = function(fd) { @@ -90,16 +91,16 @@ WriteStream.prototype._refreshSize = function() { // backwards-compat WriteStream.prototype.cursorTo = function(x, y) { - require('readline').cursorTo(this, x, y); + readline.cursorTo(this, x, y); }; WriteStream.prototype.moveCursor = function(dx, dy) { - require('readline').moveCursor(this, dx, dy); + readline.moveCursor(this, dx, dy); }; WriteStream.prototype.clearLine = function(dir) { - require('readline').clearLine(this, dir); + readline.clearLine(this, dir); }; WriteStream.prototype.clearScreenDown = function() { - require('readline').clearScreenDown(this); + readline.clearScreenDown(this); }; WriteStream.prototype.getWindowSize = function() { return [this.columns, this.rows];