search

vamolessa / pepper

simple and opinionated modal code editor for your terminal
https://vamolessa.github.io/pepper/
373 stars 15 forks source link

Windows version crashes opening a file #9

Closed wheineman closed 3 years ago

wheineman commented 3 years ago

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 commented 3 years ago

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 commented 3 years ago

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 commented 3 years ago

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 commented 3 years ago

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 commented 3 years ago

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 commented 3 years ago

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