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

Fix encountering "need-input" status in `nrepl-send-sync-request' #1070

Closed
wants to merge 1 commit into from
Closed

Conversation

Malabarba
Copy link
Member

No description provided.

@Malabarba
Copy link
Member Author

Related to #1055 and should fix #1052. But the purpose of #1055 might still be worth implementing. There may well be other cases where synchronous code asks for input.

If desired, I can merge Oleh's PR into this one, and add the proposed changes to it.

@bbatsov
Copy link
Member

bbatsov commented Apr 9, 2015

Thinking just a bit more about this - I think @cichli is right and we should think of a way to handle input properly in sync requests. Simply skipping them doesn't sound quite right.

* nrepl-client.el (nrepl-send-sync-request): On encountering
"need-input" try to call the appropriate handler or
`cider-need-input'.

* cider-debug.el (cider--debug-read-command): Be more careful with
what we try to print. If we encounter a similar problem in the future,
the user won't be faced with an error.
@Malabarba Malabarba changed the title [#1052] Append "sync" "true" to sync-request messages Fix encountering "need-input" status in `nrepl-send-sync-request' Apr 10, 2015
@Malabarba Malabarba closed this Apr 10, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants