[PATCH 3/5] d3dx9: Turn a "Out of bounds" FIXME into a WARN.

Paul Gofman gofmanp at gmail.com
Sun Jun 11 10:13:38 CDT 2017


Matteo,

     I am curious why would you want to change this FIXME to WARN? We 
know that native implementation does not fail effect creation in this case.

Regards,
     Paul.

On 06/09/2017 10:19 PM, Matteo Bruni wrote:
> Signed-off-by: Matteo Bruni <mbruni at codeweavers.com>
> ---
> It could be triggered by broken preshaders.
>
>   dlls/d3dx9_36/preshader.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/dlls/d3dx9_36/preshader.c b/dlls/d3dx9_36/preshader.c
> index 522d4841f9..d0cff22e4a 100644
> --- a/dlls/d3dx9_36/preshader.c
> +++ b/dlls/d3dx9_36/preshader.c
> @@ -868,7 +868,7 @@ static HRESULT parse_preshader(struct d3dx_preshader *pres, unsigned int *ptr, u
>               }
>               if (reg_idx >= pres->regs.table_sizes[table])
>               {
> -                FIXME("Out of bounds register index, i %u, j %u, table %u, reg_idx %u.\n",
> +                WARN("Out of bounds register index, i %u, j %u, table %u, reg_idx %u.\n",
>                           i, j, table, reg_idx);
>                   return D3DXERR_INVALIDDATA;
>               }





More information about the wine-devel mailing list