vim-fireplace icon indicating copy to clipboard operation
vim-fireplace copied to clipboard

Fix error not being reported properly

Open jessie-ross opened this issue 4 years ago • 4 comments

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:

  1. Create new deps.edn project.
  2. Add a file with a dash in it's name instead of an underscore (which every beginner is going to do 😭 ).
  3. Try to evaluate something in that file's context.
  4. See the same error as in #256

jessie-ross avatar Dec 06 '21 15:12 jessie-ross

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.

tpope avatar Dec 06 '21 16:12 tpope

Can you share the full error output?

Sure!

Screen Shot 2021-12-07 at 12 18 38 pm

With the error occuring on this line.

And this is what I see with the fix:

Screen Shot 2021-12-07 at 12 17 18 pm

jessie-ross avatar Dec 07 '21 04:12 jessie-ross

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.

jessie-ross avatar Dec 07 '21 04:12 jessie-ross

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.

tpope avatar Dec 07 '21 06:12 tpope