Am 11.03.2010 um 22:34 schrieb Roderick Colenbrander:

> Hi,
> 
> This patch makes IWineD3DDeviceImpl_ColorFill use Clear instead of
> BltOverride for color filling (it falls back to software blit if this
> is not possible). This change makes it possible in a next patch to
> have the GL Blt call IWineD3DDevice_ColorFill instead of the way
> around.
> 
> Regards,
> Roderick Colenbrander
> <0006-Implement-IWineD3DDeviceImpl_ColorFill-using-ClearSu.patch>
I am still wary of the changes to Blit because I have not seen(or have not 
understood) the overall design goal of how blits should work, and especially 
how ddraw blits and d3d8/9 style blits work together. This is especially 
visible in your patch because you still call IWineD3DBaseSurfaceImpl_Blt from 
ColorFill, you just moved the software vs gl codepath decision to a different 
place. It may be a step in the right direction, but I'd still prefer to see the 
overall strategy first.



Reply via email to