Re: [PATCH 1/5] wined3d: Fix sRGB clears with no support for ARB_framebuffer_sRGB.

[Henri Verbeet]

On 27 January 2017 at 19:52, Matteo Bruni  wrote:
> -
>    if (!gl_info->supported[ARB_FRAMEBUFFER_SRGB] && needs_srgb_write(context, state, fb))
> -
>    if (!gl_info->supported[ARB_FRAMEBUFFER_SRGB]
> -
>            && ((!(context->d3d_info->wined3d_creation_flags & WINED3D_SRGB_READ_WRITE_CONTROL)
> -
>            && fb->render_targets[0] && fb->render_targets[0]->format_flags & WINED3DFMT_FLAG_SRGB_WRITE)
> -
>            || state->render_states[WINED3D_RS_SRGBWRITEENABLE]))

I think this looks suspicious, why is this more correct?
In particular, wouldn't this enable sRGB clears on formats without
WINED3DFMT_FLAG_SRGB_WRITE? Patch 2/5 hints that that may be
intentional, but in that case, shouldn't this be independent of
ARB_framebuffer_sRGB support?