<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Oct 18, 2021 at 4:12 PM Sven Baars <<a href="mailto:sven.wine@gmail.com">sven.wine@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 18-10-2021 14:58, David Kahurani wrote:<br>
>  <br>
> +#define __WINE_DPRINTF_ONCE(dbcl,dbc2,dbch) \<br>
> +    static BOOL __wine_next_time_level;\<br>
> +    const enum __wine_debug_class __dbcl = \<br>
> +    __wine_next_time_level ? __WINE_DBCL##dbc2 : __WINE_DBCL##dbc1;\<br>
> +    (!__WINE_GET_DEBUGGING(dbcl,(dbch)) || \<br>
> +     (wine_dbg_log(__dbcl,(dbch),__FILE__,"%d: ",__LINE__) == -1)) ? \<br>
> +     (void)0 : (void)wine_dbg_printf \<br>
> +     __wine_next_time_level = TRUE;<br>
> +<br>
Hi David,<br></blockquote><div><br></div><div>Hello,</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
I don't think this will work. First because variables should be declared<br>
at the beginning of a scope, and second because if there are two<br>
FIXME_ONCE in one scope __dbcl and __wine_next_time_level will be<br>
declared twice.<br></blockquote><div><br></div><div>Should binding this into a do { } while (0) loop be enough? It looks like it should to me.</div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
I didn't check the rest of the patch, but this stood out to me.<br>
<br>
Also, Chip asked you to not send so many patches at once. Please first<br>
try to get only this patch and one of the other patches in.<br></blockquote><div><br></div><div>Well, he said in the future so my assumption was that it did not affect this patchset probably because these patches are easy enough. All the subsequent patches can be bundled into one, considered that but then again they go into different subsystems. Okay, only two patches will be sent next time.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Sven<br>
</blockquote></div></div>