[PATCH 1/4] include/wine: Introduce a FIXME_ONCE macro.

Zebediah Figura zfigura at codeweavers.com
Mon Mar 7 11:49:28 CST 2022


On 3/7/22 11:28, Alexandre Julliard wrote:
> Zebediah Figura <zfigura at codeweavers.com> writes:
> 
>> On 3/5/22 02:40, Alexandre Julliard wrote:
>>> Zebediah Figura <zfigura at codeweavers.com> writes:
>>>
>>>> +#define __WINE_DPRINTF_ONCE(dbcl1,dbcl2,dbch) \
>>>> +  do { \
>>>> +    static BOOL __wine_next_time_level;\
>>>> +    const enum __wine_debug_class __dbcl = \
>>>> +    __wine_next_time_level ? __WINE_DBCL##dbcl2 : __WINE_DBCL##dbcl1;\
>>>> +    (!__WINE_GET_DEBUGGING(dbcl1,(dbch)) || \
>>>> +     (wine_dbg_log(__dbcl,(dbch),__FILE__,"%d: ",__LINE__) == -1)) ? \
>>>> +     (void)0 : (void)wine_dbg_printf \
>>>> +     __wine_next_time_level = TRUE; } while (0)
>>> As already mentioned, this cannot possibly work.
>>>
>>
>> Sorry, that was a copy-paste error. I'm not sure where it was already
>> mentioned, though?
> 
> When the patch was originally submitted. It looks like it hasn't changed
> since then.
> 

Sorry, I managed to look for replies twice and still didn't see it.

I'll send a new version with that hunk removed.



More information about the wine-devel mailing list