Re: [PATCH v3 2/6] wined3d: Add flag to indicate FETCH4 support in textures

[Henri Verbeet]

On Sat, 24 Nov 2018 at 23:44, Daniel Ansorregui  wrote:
> @@ -340,6 +340,18 @@ static const struct wined3d_format_base_flags format_base_flags[] =
>      {WINED3DFMT_NULL,                 WINED3DFMT_FLAG_EXTENSION},
>      {WINED3DFMT_NVDB,                 WINED3DFMT_FLAG_EXTENSION},
>      {WINED3DFMT_RESZ,                 WINED3DFMT_FLAG_EXTENSION},
> - {WINED3DFMT_L8_UNORM,             WINED3DFMT_FLAG_ALLOW_FETCH4},
> - {WINED3DFMT_L16_UNORM,            WINED3DFMT_FLAG_ALLOW_FETCH4},
> - {WINED3DFMT_R16,                  WINED3DFMT_FLAG_ALLOW_FETCH4},
> - {WINED3DFMT_R32_FLOAT,            WINED3DFMT_FLAG_ALLOW_FETCH4},

One of the rules/guidelines we have in Wine is that we try to avoid
dead code. This patch runs into that because it sets a flag that isn't
used until the following patch in the series. Since both patches
aren't especially large, the best thing to do would probably be to
just merge them.
> -
> - /* The formats below are not tested to be FETCH4 compatible on windows but
> -
> * the spec "hints" that depth formats are not recomended (but allowed) */

That should be easy to test, right?