Edit buffer crashes about oni2 HOT 8 CLOSED

Wouldn't it be interesting to use another list library that doesn't throw on an empty list but returns options, so you are required to handle these cases explicitly. Even though you probably have no very good options here to handle the empty list

from oni2.

Think we could definitely do this without a new dependency, by wrapping the call in a try and returning None if Not found is raised or returning the values

from oni2.

Wouldn't a patternmatch also be a good alternative? switch (list) { | [] => ... | [hd] => ...}

from oni2.

@romgrk I can't reproduce this error on master 🤔, are you seeing or doing anything else also did you run the bootstrap command? What system are you on?

from oni2.

Yes, I've ran esy install && esy bootstrap && esy build && esy x Oni2 with the master branch just now.

$ lsb_release -a
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 18.10
Release:	18.10
$ esy --version
0.5.6

Not sure how to debug esy's ocaml, but if you have some tips I'd be happy to.

from oni2.

Thanks for logging the issue, @romgrk ! Usually this exception comes from here:

And it means that we didn't get a response from a request we sent to the Neovim API, but we were expecting one. It'd be helpful to log out the methodName in this case to see which request it is failing at (we should also have better logging in general for th is 🤔 )

We saw cases where when it picked up an init.vim and hit issues, but I don't think that is the case here.

It'd be interesting too to run the tests and see if those pass - esy x OniUnitTestRunner.

from oni2.

Yes, was just thinking about this - we should at the very least check List.length prior to List.hd (or we could create a wrapper List.hd_opt that returns an option), and provide more information here. At the very least - if it could give more details, like "Request 'nvim_open_buf' failed", it might help narrow downt he investigation.

from oni2.

@bryphe here is the result of the tests: https://gistpreview.github.io/?a07cbe32ba939e9f968e6f1ea157deb4

Logging the method names before the crash says that the last method called is nvim_list_bufs.

from oni2.