Re: [PATCH 0/8] MR10887: d3dx{10,11}: Add support for loading 2D texture array DDS files, and implement D3DX10LoadTextureFromTexture().

[Matteo Bruni (＠Mystral)]

Matteo Bruni (@Mystral) commented about dlls/d3dx10_43/texture.c:
> || (info.DstFirstElement >= dst_info.layers))
>          return S_OK;
>
> -    return E_NOTIMPL;
> +    /*
> +     * Native doesn't validate the SrcFirstMip argument, and will read OOB if
> +     * passed a value higher than the actual number of levels.
> +     */
> +    if (info.SrcFirstMip >= src_info.levels)
> +    {
> +        TRACE("Attempted to load first mip from source beyond total number of mips, clamping.\n");

I think this should be a `WARN()`, especially since native might break the application in a different way.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/10887#note_140374