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

Ignore errors when deciding whether to handle a file in global-eclim-… #242

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ovy
Copy link

@ovy ovy commented Sep 3, 2015

…mode.

(While you're looking at my other stuff, ran into this problem recently)

Otherwise a bad Eclipse state can cause an Emacs session to get very
unpleasant. I'm not sure exactly what the mechanism is, but basic stuff
like find-file, switch-buffer and the M-x prompt (of all things) stop
working with the Eclipse error.

To reproduce, define eclim--project-dir to (error something), then open a
managed Java file not already open. Have the real eclim--project-dir ready;
fortunately, eval-expression still works.

The user can run eclim-mode manually if they want to investigate the error,
or try to do anything that needs project-dir. I think even printing a
message would still be too unpleasant in that state, hence ignore.

…mode.

Otherwise a bad Eclipse state can cause an Emacs session to get very
unpleasant. I'm not sure exactly what the mechanism is, but basic stuff
like find-file, switch-buffer and *the M-x prompt* (of all things) stop
working with the Eclipse error.

To reproduce, define eclim--project-dir to (error something), then open a
managed Java file not already open. Have the real eclim--project-dir ready;
fortunately, eval-expression still works.

The user can run eclim-mode manually if they want to investigate the error,
or try to do anything that needs project-dir.  I think even printing a
message would still be too unpleasant in that state, hence ignore.
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.

1 participant