Revert "repl: handle buffered string logic on finish"

BridgeAR · BridgeAR · commit 6fed6f5e1f4a · 2018-12-07T13:32:49.000+01:00
This reverts commit eb42c1e. PR-URL: #24804 Refs: #24231 Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
diff --git a/lib/readline.js b/lib/readline.js
@@ -356,9 +356,9 @@ Interface.prototype.undoHistory = function() {
 
 // If it's a multiline code, then add history
 // accordingly.
-Interface.prototype.multilineHistory = function(clearBuffer) {
-  // if not clear buffer, add multiline history
-  if (!clearBuffer && this.terminal) {
+Interface.prototype.multilineHistory = function() {
+  // check if we got a multiline code
+  if (this.multiline !== '' && this.terminal) {
     const dupIndex = this.history.indexOf(this.multiline);
     if (dupIndex !== -1) this.history.splice(dupIndex, 1);
     // Remove the last entered line as multiline
diff --git a/lib/repl.js b/lib/repl.js
@@ -667,13 +667,6 @@ function REPLServer(prompt,
         }
       }
 
-      // handle multiline history
-      if (self[kBufferedCommandSymbol].length)
-        REPLServer.super_.prototype.multilineHistory.call(self, false);
-      else {
-        REPLServer.super_.prototype.multilineHistory.call(self, true);
-      }
-
       // Clear buffer if no SyntaxErrors
       self.clearBufferedCommand();
       sawCtrlD = false;
@@ -780,6 +773,7 @@ exports.start = function(prompt,
 
 REPLServer.prototype.clearBufferedCommand = function clearBufferedCommand() {
   this[kBufferedCommandSymbol] = '';
+  REPLServer.super_.prototype.multilineHistory.call(this);
 };
 
 REPLServer.prototype.close = function close() {
