vim-fireplace
vim-fireplace copied to clipboard
Fix error not being reported properly
This PR fixes a gap in errors being reported. Related to #256
I haven't looked at the whole code context so I am not sure if this is the best way to do so - let me know.
Replicating the issue:
- Create new deps.edn project.
- Add a file with a dash in it's name instead of an underscore (which every beginner is going to do 😭 ).
- Try to evaluate something in that file's context.
- See the same error as in #256
Can you share the full error output? E716 is pretty generic; I wouldn't jump to the conclusion that it's the same root cause as that 5 year old issue.
state.err in this context is stderr. Its presence does not necessarily warrant an exception.
Can you share the full error output?
Sure!
With the error occuring on this line.
And this is what I see with the fix:
state.err in this context is stderr. Its presence does not necessarily warrant an exception.
I am checking msg.err here, not sure if that is different, also msg.ex is set to an exception class if we want to branch on that.
I meant to say msg.err.
What are the full contents of msg.err? You can use let err = 'Clojure: ' . json_encode(msg) to check.