d3dcompiler_43: Remove a no-op statement (PVS-Studio)
Matteo Bruni
matteo.mystral at gmail.com
Wed Nov 19 08:21:22 CST 2014
2014-11-19 9:10 GMT+01:00 Michael Stefaniuc <mstefani at redhat.de>:
> D3DSRO_POSITION is 0.
Yes, it is 0, but I still feel like it should stay there for
consistency with the other cases.
What I mean is that if that line is removed I'm pretty sure I'd be
like "oh wow, this is clearly broken" every time I happen to look at
that piece of code. But maybe that's just me.
> ---
> dlls/d3dcompiler_43/bytecodewriter.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/dlls/d3dcompiler_43/bytecodewriter.c b/dlls/d3dcompiler_43/bytecodewriter.c
> index 46187f9..71ee6c1 100644
> --- a/dlls/d3dcompiler_43/bytecodewriter.c
> +++ b/dlls/d3dcompiler_43/bytecodewriter.c
> @@ -855,7 +855,6 @@ static DWORD map_vs_output(struct bc_writer *This, DWORD regnum, DWORD mask, DWO
> *has_components = TRUE;
> if(regnum == This->oPos_regnum) {
> token |= (D3DSPR_RASTOUT << D3DSP_REGTYPE_SHIFT) & D3DSP_REGTYPE_MASK;
> - token |= D3DSRO_POSITION & D3DSP_REGNUM_MASK; /* No shift */
> return token;
> }
> if(regnum == This->oFog_regnum && mask == This->oFog_mask) {
> --
> 1.9.3
Cheers,
Matteo.
More information about the wine-devel
mailing list