-
Notifications
You must be signed in to change notification settings - Fork 247
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
Adds a buffer before the backend #786
Conversation
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as off-topic.
This comment was marked as off-topic.
match width { | ||
1 => CellWidth::Single, | ||
2 => CellWidth::Double, | ||
_ => panic!("expected width of 1 or 2 only."), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can that be this:
+ n => panic!("expected width of 1 or 2 only, got {n}."),
It shows got 0.
in case of \0
(nuls) in string.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Indeed, I'm now skipping zero-width graphemes entirely.
This comment was marked as outdated.
This comment was marked as outdated.
...to handle just-in PR gyscos/cursive#786
Just realized it was running the tests from the |
This is amazing work.
I feel like it's worth noting that some tests will be missed, but maybe this is intentional? that's why I suggested more than 1 command(but maybe they can be merged, didn't look into it) For example:
|
Interesting... the difference seems to be that it looks for unit tests in the examples? Running Is there any error the first command would not catch? |
no errors currently, but I figured maybe it's worth not skipping those extra tests, there are at least(apparently the only?) these 2 that the first one is missing:
So there are far less tests that are missing that I initially thought, 2 out of 28, the rest are overlapping with |
Oh ha ha I completely forgot we had this |
This accumulates print operations in a draw cycle, and flushes the result out to the backend at the end.
It also computes a delta and only flushes the changes, reducing the amount of IO, and improving performance for the termion and crossterm backends (the ncurses/pancurses backend was already doing that internally).