Fix error checking in dlls/ddraw/executebuffer.c

Paul Vriens paul.vriens.wine at gmail.com
Sat Jun 20 07:07:49 CDT 2009


Gerald Pfeifer wrote:
> I believe, after looking at the ranges of D3DLIGHTSTATETYPE, that
> the below is the actual intention of this code (to ensure we are
> in range).
> 
> Gerald
> 
> ChangeLog:
> Fix error checking in IDirect3DExecuteBufferImpl_Execute().
> 
> diff --git a/dlls/ddraw/executebuffer.c b/dlls/ddraw/executebuffer.c
> index 73e0657..6d6d304 100644
> --- a/dlls/ddraw/executebuffer.c
> +++ b/dlls/ddraw/executebuffer.c
> @@ -249,7 +249,7 @@ IDirect3DExecuteBufferImpl_Execute(IDirect3DExecuteBufferImpl *This,
>  
>  		    TRACE("(%08x,%08x)\n", ci->u1.dlstLightStateType, ci->u2.dwArg[0]);
>  
> -		    if (!ci->u1.dlstLightStateType && (ci->u1.dlstLightStateType > D3DLIGHTSTATE_COLORVERTEX))
> +		    if (!ci->u1.dlstLightStateType || (ci->u1.dlstLightStateType > D3DLIGHTSTATE_COLORVERTEX))
>  			ERR("Unexpected Light State Type\n");

Would:

if ((ci->u1.dlstLightStateType < D3DLIGHTSTATE_MATERIAL) || 
(ci->u1.dlstLightStateType > D3DLIGHTSTATE_COLORVERTEX))

be easier to read? (Matter of taste I guess).

-- 
Cheers,

Paul.



More information about the wine-devel mailing list