Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

repl: add completion preview #30907

Closed
wants to merge 12 commits into from
Closed

repl: add completion preview #30907

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
19dac98
repl: remove dead code
BridgeAR Dec 10, 2019
c65f3f1
repl: simplify repl autocompletion
BridgeAR Dec 10, 2019
ec47482
repl: simplify code
BridgeAR Dec 10, 2019
43ae689
readline: update ansi-regex
BridgeAR Dec 11, 2019
1426e6c
repl,readline: refactor common code
BridgeAR Dec 11, 2019
42d3590
repl,readline: refactor for simplicity
BridgeAR Dec 11, 2019
ca14259
repl: improve completion
BridgeAR Dec 11, 2019
1cbef42
repl: add completion preview
BridgeAR Dec 11, 2019
707cba4
repl: fix preview bug in case of long lines
BridgeAR Dec 11, 2019
ca4c894
test: add multiple repl preview tests
BridgeAR Dec 13, 2019
0cd4153
fixup: repl: add completion preview
BridgeAR Dec 14, 2019
2138fd7
fixup: test: add multiple repl preview tests
BridgeAR Dec 14, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
repl: simplify repl autocompletion 
This simplifies calling `filteredOwnPropertyNames()`. The context
is not used in that function, so there's no need to call the function
as such.
  • Loading branch information
@BridgeAR
BridgeAR committed Dec 14, 2019
commit c65f3f1384865ceff9dc08db5c86b4de47aaf02f
10 changes: 4 additions & 6 deletions lib/repl.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1280,11 +1280,9 @@ function complete(line, callback) {
completionGroups.push(getGlobalLexicalScopeNames(this[kContextId]));
let contextProto = this.context;
while (contextProto = ObjectGetPrototypeOf(contextProto)) {
completionGroups.push(
filteredOwnPropertyNames.call(this, contextProto));
completionGroups.push(filteredOwnPropertyNames(contextProto));
}
const contextOwnNames =
filteredOwnPropertyNames.call(this, this.context);
const contextOwnNames = filteredOwnPropertyNames(this.context);
if (!this.useGlobal) {
// When the context is not `global`, builtins are not own
// properties of it.
Expand All @@ -1299,7 +1297,7 @@ function complete(line, callback) {
if (obj != null) {
if (typeof obj === 'object' || typeof obj === 'function') {
try {
memberGroups.push(filteredOwnPropertyNames.call(this, obj));
memberGroups.push(filteredOwnPropertyNames(obj));
} catch {
// Probably a Proxy object without `getOwnPropertyNames` trap.
// We simply ignore it here, as we don't want to break the
Expand All @@ -1317,7 +1315,7 @@ function complete(line, callback) {
p = obj.constructor ? obj.constructor.prototype : null;
}
while (p !== null) {
memberGroups.push(filteredOwnPropertyNames.call(this, p));
memberGroups.push(filteredOwnPropertyNames(p));
p = ObjectGetPrototypeOf(p);
// Circular refs possible? Let's guard against that.
sentinel--;
Expand Down