After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 416635 - Rendering glitch: double draw of line below exposed region
Rendering glitch: double draw of line below exposed region
Status: RESOLVED FIXED
Product: vte
Classification: Core
Component: general
0.15.x
Other Linux
: Normal major
: ---
Assigned To: VTE Maintainers
VTE Maintainers
Depends on:
Blocks:
 
 
Reported: 2007-03-10 00:44 UTC by Chris Wilson
Modified: 2007-03-16 10:19 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Correct the redundant overdraw, since we now explicitly expand the regions before paint_area() (865 bytes, patch)
2007-03-12 21:50 UTC, Chris Wilson
none Details | Review
Correct the redundant overdraw, since we now explicitly expand the regions before paint_area() (1.74 KB, patch)
2007-03-15 15:14 UTC, Chris Wilson
none Details | Review

Description Chris Wilson 2007-03-10 00:44:07 UTC
Lest I forget, Behdad reported this one as well...

Occassionally the prompt may rendered bold after an ls - the occurrence is highly dependent on the speed of the input.
Comment 1 Behdad Esfahbod 2007-03-10 18:42:35 UTC
The boldness looks to be a result of rendering again without clearing the background first.
Comment 2 Chris Wilson 2007-03-10 22:58:30 UTC
Good eyes, Behdad. It definitely appears to be another double draw issue. The tricky part here was to have multiple neighbouring exposed regions, then a slight OBO in paint_area() caused the first paint_area() to render characters belonging to the second paint_area(). [Allowed through the clip mask, since we only create a single mask covering the entire invalidated area. And it is only showing up now as I introduced a patch to reduce the number of redundant clears.]

Yet another patch to split out of the queue...
Comment 3 Chris Wilson 2007-03-12 21:50:46 UTC
Created attachment 84458 [details] [review]
Correct the redundant overdraw, since we now explicitly expand the regions before paint_area()
Comment 4 Chris Wilson 2007-03-15 15:14:41 UTC
Created attachment 84656 [details] [review]
Correct the redundant overdraw, since we now explicitly expand the regions before paint_area()

The previous patch exposed a bug in the enlargement that should have been taking place in vte_terminal_expand_region()
Comment 5 Chris Wilson 2007-03-16 10:19:13 UTC
r1838: 2007-03-16  Chris Wilson  <chris@chris-wilson.co.uk>

	Bug 416635 – Rendering glitch: double draw of line below exposed region

	* src/vte.c (vte_terminal_expand_region),
	(vte_terminal_paint_area): Kill a couple of off-by-ones.