Re: [PATCH v4 0/2] MR979: d3d9: Implement d3dx_effect_SetRawValue

[Matteo Bruni (＠Mystral)]

Matteo Bruni (@Mystral) commented about dlls/d3dx9_36/effect.c:
> - struct d3dx_parameter *param = get_valid_parameter( src, parameter );
> - void *raw = NULL;
> -
> - TRACE("iface %p, parameter %p, data %p, byte_offset %u, bytes %u.\n",
>         iface, parameter, data, byte_offset, bytes);
> - return E_NOTIMPL;
> - if (!src)
> - {
> -
>    WARN("Invalid variable.\n");
> -
>    return E_FAIL;
> - }
> -
> - if (!data || !param || !(raw = param_get_data_and_dirtify( src, param,
> -
>        byte_offset + bytes, TRUE )))
> -
>    return D3DERR_INVALIDCALL;

param_get_data_and_dirtify() can't really return NULL.