Skip to content

Conversation

@kchibisov
Copy link
Member

When terminal should be fully damaged we only want to occupied cells,
thus now alacritty_terminal always returns TermDamageIterator, but
when terminal was fully damaged it damages all the lines from 0 up to
line.occ. That helps a lot with something like yes or when scrolling
scrollback buffer by reducing the amount of damaged area.

@kchibisov kchibisov force-pushed the optimize-full-damage branch 3 times, most recently from 1709a7f to 241c8ce Compare February 5, 2022 04:22
@kchibisov kchibisov requested a review from chrisduerr February 5, 2022 04:23
@kchibisov kchibisov force-pushed the optimize-full-damage branch from 477b3a7 to 037ba09 Compare February 5, 2022 18:24
@alacritty alacritty deleted a comment from perfbot Feb 5, 2022
@alacritty alacritty deleted a comment from perfbot Feb 5, 2022
When terminal should be fully damaged we only want to occupied cells,
thus now alacritty_terminal always returns `TermDamageIterator`, but
when terminal was fully damaged it damages all the lines from 0 up to
line.occ. That helps a lot with something like `yes` or when scrolling
scrollback buffer by reducing the amount of damaged area.
@kchibisov kchibisov force-pushed the optimize-full-damage branch from a02bca6 to 480ba15 Compare February 6, 2022 18:12
@chrisduerr
Copy link
Member

@perfbot

@perfbot
Copy link

perfbot commented Feb 6, 2022

results

@alacritty alacritty deleted a comment from perfbot Feb 6, 2022
@alacritty alacritty deleted a comment from perfbot Feb 6, 2022
@alacritty alacritty deleted a comment from perfbot Feb 6, 2022
@alacritty alacritty deleted a comment from perfbot Feb 6, 2022
@alacritty alacritty deleted a comment from perfbot Feb 6, 2022
@alacritty alacritty deleted a comment from perfbot Feb 6, 2022
@alacritty alacritty deleted a comment from perfbot Feb 6, 2022
@kchibisov
Copy link
Member Author

Closing in favor of #5863.

@kchibisov kchibisov closed this Feb 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants