panic when pressing 'space' in empty direcotry about lf HOT 5 CLOSED

Well, obviously we're not handling the case of selecting a file if there isn't any.
I wonder where this should be handled, though. Probably the cleanest (conceptually) solution would be for currPath (and currFile?) to return error in such case. But I'm afraid this will quickly turn annoying in practice.

from lf.

I haven't read the codebase so I can't help you much with this. I just created this issue because I don't think its good to panic and kill the program in any circumstances. And this happens on simple key press.

from lf.

You're absolutely right @adamjedlicka . Panic is definitely not acceptable for these things. We only panic for major exceptions in a few places in lf (e.g. initialize termbox). In this case it's an array out of bounds exception by the runtime. So we can't avoid it but only fix it. There were a few such errors previously for empty directories so I spent some time thinking about how to refactor the code to avoid these errors but I couldn't find an easy way. These should be rare though and we should get stable in time. I will try to take care of this as soon as possible. Thanks for the report.

from lf.

Sure, @adamjedlicka, thanks for taking your time to report. My comment was intended to whoever is willing to work on it, to know better what's going on in code.
I will try to get back to #20 in the coming days, hopefully I'll scratch this one on the way as well.

from lf.

@adamjedlicka With the latest commit toggling and yanking in an empty directory doesn't seem to cause any trouble. Closing this issue now. Feel free to open again if you continue to have problems after you update to the latest version.

from lf.