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.

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: tab completion regression in v6 #6325

Closed
targos opened this issue Apr 21, 2016 · 1 comment
Closed

repl: tab completion regression in v6 #6325

targos opened this issue Apr 21, 2016 · 1 comment
Labels
repl Issues and PRs related to the REPL subsystem.

Comments

@targos
Copy link
Member

targos commented Apr 21, 2016

  • Version: v6.0.0-rc.3
  • Platform:
  • Subsystem: repl

It seems that tab completion only works if the element to be completed is at the beginning of the line.
Examples:

  • > buf<TAB> completes to buffer
  • > <SPACE>buf<TAB> does not complete
  • > buffer.<TAB> completes to the list of buffer properties
  • > var buf = buffer.Slow<TAB> does not complete
@targos targos added the repl Issues and PRs related to the REPL subsystem. label Apr 21, 2016
@addaleax
Copy link
Member

That seems to be my #6192, sorry… I’m on it.

addaleax added a commit to addaleax/node that referenced this issue Apr 25, 2016
Instead of changing the way “simple” expressions are detected,
switch to ignoring errors when completing. This approach is more
generic than the previous one from 0b66b8f, but also changes
the way errors are thrown when completing.

This reverts the code changes from commit 0b66b8f.
The test case is left intact.

Fixes: nodejs#6325
PR-URL: nodejs#6328
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
jasnell pushed a commit that referenced this issue Apr 26, 2016
Instead of changing the way “simple” expressions are detected,
switch to ignoring errors when completing. This approach is more
generic than the previous one from 0b66b8f, but also changes
the way errors are thrown when completing.

This reverts the code changes from commit 0b66b8f.
The test case is left intact.

Fixes: #6325
PR-URL: #6328
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
repl Issues and PRs related to the REPL subsystem.
Projects
None yet
Development

No branches or pull requests

2 participants