Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

byte index 51 is not a char boundary #15

Closed
mosubpink opened this issue Apr 30, 2024 · 5 comments
Closed

byte index 51 is not a char boundary #15

mosubpink opened this issue Apr 30, 2024 · 5 comments

Comments

@mosubpink
Copy link

nyaa not runnig after i update it to last verison it gave the error :

thread 'main' panicked at /home/user/.cargo/registry/src/index.crates.io-6f17d22bba15001f/ratatui-0.26.2/src/text/line.rs:477:59:
                              byte index 51 is not a char boundary; it is inside 'о' (bytes 50..52) of `Sentai Dai Shikkaku | Боевой отряд «Полный провал» | Go! Go! Loser Ranger! [ТВ-1] [2024] [04 of 12] [WEBRip] [1080p] [RUS + JAP]`
                                 note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Screenshot from 2024-04-30 19-49-25

@Beastwick18
Copy link
Owner

I'm not able to replicate this on v0.8.0. This is the version you are running right? You can check with nyaa -V. Does this happen every time the program is launched?

@Beastwick18
Copy link
Owner

Ah nevermind, I was able to replicate it, but I'm not sure of whats causing it. I'll look into this, but it seems to not happen on the up to date main branch. So whatever it was might already be fixed, but unreleased.

@Beastwick18
Copy link
Owner

Did some further digging, looks like this is an upstream issue with ratatui: ratatui/ratatui#1032

I'll push a fix soon, hopefully all that has to be done is to downgrade ratatui until it gets fixed upstream.

@Beastwick18
Copy link
Owner

This is now fixed on main, and the fix will be in the next release. In the meantime, if you want to build with cargo to get the fix earlier you can use:

cargo install --git https://github.com/Beastwick18/nyaa

Make sure you have rust version >=1.75 installed first.

@Beastwick18
Copy link
Owner

Fixed in v0.8.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants