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