Skip to content

repl: add completion preview #30907

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

Closed
BridgeAR 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 code 
This simplifies some repl code and removes a coe branch that is
unreachable.
  • Loading branch information
@BridgeAR
BridgeAR committed Dec 14, 2019
commit ec47482391ecfedfd0d910f03658e002789d3515
12 changes: 4 additions & 8 deletions lib/repl.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -953,7 +953,7 @@ REPLServer.prototype.createContext = function() {
}

const module = new CJSModule('<repl>');
module.paths = CJSModule._resolveLookupPaths('<repl>', parentModule) || [];
module.paths = CJSModule._resolveLookupPaths('<repl>', parentModule);

ObjectDefineProperty(context, 'module', {
configurable: true,
Expand Down Expand Up @@ -1307,21 +1307,17 @@ function complete(line, callback) {
}
// Works for non-objects
try {
let sentinel = 5;
let p;
if (typeof obj === 'object' || typeof obj === 'function') {
p = ObjectGetPrototypeOf(obj);
} else {
p = obj.constructor ? obj.constructor.prototype : null;
}
while (p !== null) {
// Circular refs possible? Let's guard against that.
let sentinel = 5;
while (p !== null && sentinel-- !== 0) {
memberGroups.push(filteredOwnPropertyNames(p));
p = ObjectGetPrototypeOf(p);
// Circular refs possible? Let's guard against that.
sentinel--;
if (sentinel <= 0) {
break;
}
}
} catch {}
}
Expand Down