til::bitmap for dxengine should just be X=1 if we're painting one full row #9053

Closed
opened 2026-01-31 01:44:47 +00:00 by claunia · 4 comments
Owner

Originally created by @miniksa on GitHub (Jun 15, 2020).

There's a lot of wasted work twiddling bits if we're setting an entire row worth of them at the time. Which we are to ensure ligatures work. That's fine, but let's save ourselves the bit twiddling time.

Originally created by @miniksa on GitHub (Jun 15, 2020). There's a lot of wasted work twiddling bits if we're setting an entire row worth of them at the time. Which we are to ensure ligatures work. That's fine, but let's save ourselves the bit twiddling time.
Author
Owner

@DHowett commented on GitHub (Jul 10, 2020):

Moved into 2.0. @miniksa if this didn't have the impact we were hoping, feel free to close.

@DHowett commented on GitHub (Jul 10, 2020): Moved into 2.0. @miniksa if this didn't have the impact we were hoping, feel free to close.
Author
Owner

@miniksa commented on GitHub (Jul 10, 2020):

Moved into 2.0. @miniksa if this didn't have the impact we were hoping, feel free to close.

I'll try again before discarding this. I might have been taking crazy pills the day I didn't see a change.

@miniksa commented on GitHub (Jul 10, 2020): > Moved into 2.0. @miniksa if this didn't have the impact we were hoping, feel free to close. I'll try again before discarding this. I might have been taking crazy pills the day I didn't see a change.
Author
Owner

@zadjii-msft commented on GitHub (Sep 7, 2023):

@lhecker this relevant for Atlas engine? If it's not, I'm tempted to close it out as "the DxEngine is less than maintained"

@zadjii-msft commented on GitHub (Sep 7, 2023): @lhecker this relevant for Atlas engine? If it's not, I'm tempted to close it out as "the DxEngine is less than maintained"
Author
Owner

@lhecker commented on GitHub (Sep 7, 2023):

It's not relevant. AtlasEngine doesn't just invalidate entire rows at a time, but holds just a single [begin, end) range of rows to invalidate. If you modify row 0 and N-1 then the entire viewport is invalidated (but that's fine because it's seldom and still fast).

@lhecker commented on GitHub (Sep 7, 2023): It's not relevant. AtlasEngine doesn't just invalidate entire rows at a time, but holds just a single `[begin, end)` range of rows to invalidate. If you modify row 0 and N-1 then the entire viewport is invalidated (but that's fine because it's seldom and still fast).
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#9053