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

code --add breaks if executed repeatedly #38138

Closed
fabiospampinato opened this issue Nov 12, 2017 · 4 comments
Closed

code --add breaks if executed repeatedly #38138

fabiospampinato opened this issue Nov 12, 2017 · 4 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded workbench-multiroot Multi-root (multiple folders) issues
Milestone

Comments

@fabiospampinato
Copy link
Contributor

fabiospampinato commented Nov 12, 2017

  • VSCode Version: Code 1.18.0
  • OS Version: Darwin x64 17.3.0

Executing a command like code --add path1 && code --add path2 (...) code --add pathN completely breaks the window, I only get a couple of paths added no matter what, duplicated icons in the statusbar and the entire thing basically becomes unstable/unusable.

Executing code --add path1 path2 (...) pathN works just fine, unfortunately for a high number of paths I can't submit this command to the terminal via the terminal#sendText method because of #38137.

@bpasero bpasero self-assigned this Nov 12, 2017
@bpasero bpasero added bug Issue identified by VS Code Team member as probable bug workbench-multiroot Multi-root (multiple folders) issues labels Nov 12, 2017
@bpasero bpasero added this to the November 2017 milestone Nov 12, 2017
@bpasero
Copy link
Member

bpasero commented Nov 12, 2017

@fabiospampinato what exactly happens in your case? For me I just see the last folder not getting added (only 2 work) and an exception in the console. Do you also see an exception?

@fabiospampinato
Copy link
Contributor Author

@bpasero I've made a screencast to properly illustrate this.

Basically: I only get 2 folders, no exceptions, duplicated icons in the statusbar and the window basically freezes for a while.

@bpasero bpasero removed this from the November 2017 milestone Nov 12, 2017
@bpasero bpasero added the info-needed Issue requires more information from poster label Nov 12, 2017
@fabiospampinato
Copy link
Contributor Author

@bpasero do you need "more info" from me? If so, what else is needed?

@bpasero bpasero removed the info-needed Issue requires more information from poster label Nov 14, 2017
@bpasero bpasero added this to the November 2017 milestone Nov 14, 2017
@bpasero
Copy link
Member

bpasero commented Nov 14, 2017

This is a race condition where the first --add request causes lots of work to happening. A pushed a change to buffer the requests for --add for 100ms before processing them. I think that should solve the issue.

@mjbvz mjbvz added the verified Verification succeeded label Dec 6, 2017
@vscodebot vscodebot bot locked and limited conversation to collaborators Dec 29, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded workbench-multiroot Multi-root (multiple folders) issues
Projects
None yet
Development

No branches or pull requests

3 participants