CVS Update: xc (branch: trunk)

[Luc Verhaegen]

On Tue, Oct 25, 2005 at 10:18:30AM -0700, Thomas Hellstrom wrote:
> CVSROOT:	/cvs/xorg
> Module name:	xc
> Changes by:	unichrome at gabe.freedesktop.org	05/10/25 10:18:30
> 
> Log message:
>   	* programs/Xserver/hw/xfree86/drivers/via/via_video.c:
>   	(viaPutImage):
>   	Colorkey autopainting bugfix
> 
> Modified files:
>       ./:
>         ChangeLog 
>       xc/programs/Xserver/hw/xfree86/drivers/via/:
>         via_video.c 
>   
>   Revision      Changes    Path
>   1.1474        +6 -0      xc/ChangeLog
>   http://cvs.freedesktop.org/xorg/xc/ChangeLog
>   1.14          +3 -5      xc/programs/Xserver/hw/xfree86/drivers/via/via_video.c
>   http://cvs.freedesktop.org/xorg/xc/programs/Xserver/hw/xfree86/drivers/via/via_video.c
> 
I'm curious. What bug does this fix? What are you seeing?

Surely the colorkey doesn't need to be written to the FB each time, say, 
only a new frame is sent (which is undoubtedly the most common case). 
When focused on cpu usage, that's quite a few cycles you can spare right 
there.

Why remove it in PutImage and keep it in ReputImage?

If initialisation is what you're worried about, StopVideo and 
SetupAdaptors seem to take care of that.

Luc Verhaegen.