Commit ed83202
repl: remove obsolete completer variable
It is also assigned in readline. There is not need to assign the
variable twice.
Signed-off-by: Ruben Bridgewater <ruben@bridgewater.de>
PR-URL: #33294
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>1 parent db7bb94 commit ed83202
1 file changed
Lines changed: 1 addition & 5 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -647,10 +647,6 @@ function REPLServer(prompt, | |||
| 647 | 647 | enumerable: true | |
| 648 | 648 | }); | |
| 649 | 649 | ||
| 650 | - // Figure out which "complete" function to use. | ||
| 651 | - self.completer = (typeof options.completer === 'function') ? | ||
| 652 | - options.completer : completer; | ||
| 653 | - | ||
| 654 | 650 | function completer(text, cb) { | |
| 655 | 651 | complete.call(self, text, self.editorMode ? | |
| 656 | 652 | self.completeOnEditorMode(cb) : cb); | |
@@ -659,7 +655,7 @@ function REPLServer(prompt, | |||
| 659 | 655 | Interface.call(this, { | |
| 660 | 656 | input: options.input, | |
| 661 | 657 | output: options.output, | |
| 662 | - completer: self.completer, | ||
| 658 | + completer: options.completer || completer, | ||
| 663 | 659 | terminal: options.terminal, | |
| 664 | 660 | historySize: options.historySize, | |
| 665 | 661 | prompt | |
0 commit comments