-
Notifications
You must be signed in to change notification settings - Fork 38
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
documentation shouldn't err or "" #35
Comments
More info - clojure-emacs/cider-nrepl#294 |
Agreed. Although I would expect CIDER middleware to catch all possible underlying exceptions and send the trace via its internal pipes. |
That was just fixed in the middleware. |
OK. So should I still put a try block around that code? In that case I would just silently swallow exceptions which would make debugging more difficult. Another thing to do is just to handle empty input to |
I'd handle empty empty input. Seems like the reasonable thing to do. |
Fixed in e6040df. |
Ideally this should return "" (what happens with unknown symbols).
The text was updated successfully, but these errors were encountered: