Christian Brabandt <[email protected]> wrote: > Bram, > while debugging the xdiff test, I stumbled about the error message > E685: Internal error: set_var() > > That was me, trying to clear v:errors, but unfortunately I thought > v:errors are of type string, while it is a list type. So the first time > :let v:errors='' > failed with the above error, while on the next try, it actually > succeeded. That is actually a bit confusing and just after hunting the > set_var() message, I actually found out what I did wrong :( > > So can we please make this something more obvious, e.g. like this: > > diff --git a/src/eval.c b/src/eval.c > index 7dfb98125..dd0164f04 100644 > --- a/src/eval.c > +++ b/src/eval.c > @@ -7568,7 +7568,10 @@ set_var( > return; > } > else if (v->di_tv.v_type != tv->v_type) > - internal_error("set_var()"); > + { > + internal_error("set_var(): wrong type!"); > + return; > + } > } > > clear_tv(&v->di_tv);
Any internal error is a bug, so I don't think that let v:errors='' should give an internal error, it should give a regular error. Dominique -- -- You received this message from the "vim_dev" maillist. Do not top-post! Type your reply below the text you are replying to. For more information, visit http://www.vim.org/maillist.php --- You received this message because you are subscribed to the Google Groups "vim_dev" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. For more options, visit https://groups.google.com/d/optout.
