[PATCH 02/11] wined3d: Add WINED3DSWAPCHAIN_ALLOW_MODE_SWITCH flag.

Józef Kucia joseph.kucia at gmail.com
Tue Aug 2 06:11:36 CDT 2016


On Tue, Aug 2, 2016 at 12:45 PM, Henri Verbeet <hverbeet at gmail.com> wrote:
> On 1 August 2016 at 22:28, Józef Kucia <jkucia at codeweavers.com> wrote:
>> +        if (mode)
>> +            ERR("Ignoring mode.\n");
> I think it's unclear what this means. "Ignoring mode" seems to suggest
> we're ignoring the mode, but shouldn't, in which case this should be a
> FIXME instead of an ERR. It could also mean the user of the API
> supplied a mode, but it has no effect because
> WINED3DSWAPCHAIN_ALLOW_MODE_SWITCH is not set on the swapchain, in
> which case this should be a WARN instead of an ERR.

Generally, we never call this function with a mode when
WINED3DSWAPCHAIN_ALLOW_MODE_SWITCH is not set. It should be a WARN
anyway.



More information about the wine-devel mailing list