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.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/979#note_11084