-
Notifications
You must be signed in to change notification settings - Fork 31
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
Panic from print_text
#22
Comments
I do reproduce it. Smaller example:
|
This should have been fixed. Can you please confirm it's OK @samtay ? |
Yes, I just tried out |
lerina
pushed a commit
to lerina/termimad
that referenced
this issue
Jan 14, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Someone has observed
thread 'main' panicked at 'attempt to subtract with overflow'
coming from askin.print_text
call (ref samtay/so#8), and also found that such panic goes away when usingskin.print_inline
.Unfortunately I couldn't repo this myself, but the relevant details from that issue are:(190, 51)
Note the markdown input is quite large, so I put it in a gist. I also embedded it within the
./examples/text/main.rs
file so that it would be easier to run.EDIT: I was able to repro this. At terminal width
112
there is no panic. If the terminal width is greater than or equal to117
, you can observe this panic.I think if you make sure your terminal width is that wide (e.g. go full screen), and run this:
you should be able to observe it.
The text was updated successfully, but these errors were encountered: