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

Optimize terminal full damage #5858

Closed
wants to merge 2 commits into from

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
@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.
@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.

None yet

3 participants