Re: [PATCH v2 0/6] MR1682: d3dx9: Partially implement texture filling using texture shaders.

[Matteo Bruni (＠Mystral)]

Matteo Bruni (@Mystral) commented about dlls/d3dx9_36/tests/texture.c:
> +        {XCOORD, ZERO, YCOORDINV},
> +        {XCOORD, YCOORDINV, ONE},
> +        {XCOORDINV, YCOORDINV, ZERO}
> +    };
>
> -    if (SUCCEEDED(hr))
> +    DWORD x, y, m, f, levels, size, value, expected;
> +    D3DXVECTOR3 coord, texelsize;
> +    D3DLOCKED_RECT lock_rect;
> +    D3DSURFACE_DESC desc;
> +    D3DXVECTOR4 out;
> +    HRESULT hr;
> +
> +    levels = IDirect3DCubeTexture9_GetLevelCount(texture);
> +
> +    for (m = 0; m < levels; m++)
> Given that you're touching these lines anyway, you can probably replace the postincrement operator with preincrement (we use the latter by default).

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/1682#note_19499