Ticket #10695 (closed Patch - Bug Fix: fixed)
Opened 13 months ago
Last modified 7 months ago
[PATCH] libmythui: Fix a redraw bug in MythUIType
|Reported by:||Lawrence Rust <lvr@…>||Owned by:||stuartm|
|Component:||MythTV - General||Version:||Master Head|
|Severity:||medium||Keywords:||keypress redraw tearfree|
While evaluating the ATI Catalyst driver 'TearFree? Desktop' option I noticed that sometimes a keypress was apparently being held up until the time on the main menu was updated. Initially I thought that this was a lirc or fglrx issue, but on investigation it turned out that MythMainWindow::animate was requesting updates for empty repaint regions. Delving further showed that MythUIType::Draw was resetting m_DirtyRegion to empty even if the clipRect was smaller, thus leaving unpainted regions.
m_DirtyRegion may be extended by HandleMovementPulse?, SetRedraw? or SetChildNeedsRedraw? etc AFTER GetDirtyArea? is called. So when MythUIType::Draw is called, clipRect may not include the whole of m_DirtyRegion.
This patch subtracts the Draw clipRect from m_DirtyRegion instead of simply emtying it.
Changed 13 months ago by Lawrence Rust <lvr@…>
- Attachment 0001-libmythui-Fix-a-redraw-bug-in-MythUIType.patch added
comment:1 Changed 11 months ago by beirdo
- Owner set to stuartm
- Status changed from new to assigned
comment:2 Changed 7 months ago by stuartm
- Status changed from assigned to accepted
- Milestone changed from unknown to 0.27
comment:3 Changed 7 months ago by Lawrence Rust <lvr@…>
- Status changed from accepted to closed
- Resolution set to fixed