[PATCH v2 4/7] ucrtbase: Implement the new printf corner case behaviour

Martin Storsjö martin at martin.st
Tue Nov 3 03:40:48 CST 2015

On Tue, 3 Nov 2015, Piotr Caban wrote:

> On 11/02/15 22:26, Martin Storsjo wrote:
>> -        FIXME("options %s not handled\n", wine_dbgstr_longlong(options));
> Shouldn't this be changed to:
> if(options & ~UCRTBASE_PRINTF_MASK)
>    FIXME("...);
> instead of being removed?

Perhaps, although these are all the flags that are currently defined, so 
as far as I can see, there's nothing left to warn about (unless someone 
calls this internal function deliberately with weird/undefined flags). 
(The mask when caling pf_printf is only to make sure that we don't 
accidentally alias the flags of our own.)

Should we keep a warning in case a future version adds more flags?

// Martin

More information about the wine-devel mailing list