Windows version crashes opening a file

[wheineman]

Cargo built version v0.10.6 crashes when opening a file, either from the command line or with the open command:
thread '' panicked at 'index out of bounds: the len is 0 but the index is 0', C:\Users\whein.cargo\registry\src\github.com-1ecc6299db9ec823\pepper-0.10.6\src\buffer.rs:364:10

[vamolessa]

What were the file contents?
You can reproduce it just by invoking pepper filename.ext with default configs?
Does this happen with any file or just one specifically?
Did you built from source? If so, could you try building from master's current HEAD b1657c5?

[wheineman]

Hi,    It happens with any text file. I created a text file call test.txt with one line in it and it failed to load. It failed when I invoked "pepper test.txt" with default configs (I have not created any config file).I built and installed pepper using cargo, i.e. "cargo install pepper".I will try building from the HEAD and let you know the results. Cheers, Willy On Friday, May 28, 2021, 9:51:19 AM EDT, Matheus Lessa Rodrigues ***@***.***> wrote: What were the file contents? You can reproduce it just by invoking pepper filename.ext with default configs? Does this happen with any file or just one specifically? Did you built from source? If so, could you try building from master's current HEAD b1657c5? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

[vamolessa]

Did it work using a newer commit? I really could not reproduce your issue.
I develop it on windows and use pepper as my main editor but this crash never happened to me.
If you have any further clue, please let me know

[wheineman]

Hi,    I cloned the github repository and built both the debug and release versions of Pepper and they both read files fine, whether from the command line at startup or with the open command. So that's great. I don't know why the one built with the "cargo install pepper" command failed but maybe that should be pointed out for Windows users. Cheers, Willy On Saturday, May 29, 2021, 11:40:30 AM EDT, Matheus Lessa Rodrigues ***@***.***> wrote: Did it work using a newer commit? I really could not reproduce your issue. I develop it on windows and use pepper as my main editor but this crash never happened to me. If you have any further clue, please let me know — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

[wheineman]

Also thanks for pepper. I'm enjoying it immensely. Cheers, Willy On Saturday, May 29, 2021, 3:27:34 PM EDT, William Heineman ***@***.***> wrote: Hi,    I cloned the github repository and built both the debug and release versions of Pepper and they both read files fine, whether from the command line at startup or with the open command. So that's great. I don't know why the one built with the "cargo install pepper" command failed but maybe that should be pointed out for Windows users. Cheers, Willy On Saturday, May 29, 2021, 11:40:30 AM EDT, Matheus Lessa Rodrigues ***@***.***> wrote: Did it work using a newer commit? I really could not reproduce your issue. I develop it on windows and use pepper as my main editor but this crash never happened to me. If you have any further clue, please let me know — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

[vamolessa]

Ah, that's good to hear! Thanks for following up!
Gonna close this then