Fix error checking in dlls/ddraw/executebuffer.c

Gerald Pfeifer gerald at pfeifer.com
Sat Jun 20 10:57:09 CDT 2009


On Sat, 20 Jun 2009, Paul Vriens wrote:
>>  -		    if (!ci->u1.dlstLightStateType && (ci->u1.dlstLightStateType > D3DLIGHTSTATE_COLORVERTEX))
>> +		    if (!ci->u1.dlstLightStateType || (ci->u1.dlstLightStateType > D3DLIGHTSTATE_COLORVERTEX))
> Would:
> 
> if ((ci->u1.dlstLightStateType < D3DLIGHTSTATE_MATERIAL) ||
> (ci->u1.dlstLightStateType > D3DLIGHTSTATE_COLORVERTEX))
> 
> be easier to read? (Matter of taste I guess).

I found the existing check (with the bug fixed ;-) easier to understand, 
but as you say it's a matter of taste and I do not feel strongly about it.

Gerald



More information about the wine-devel mailing list