<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">2013/5/27 Henri Verbeet <span dir="ltr"><<a href="mailto:hverbeet@gmail.com" target="_blank">hverbeet@gmail.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On 27 May 2013 09:49, Christian Costa <<a href="mailto:titan.costa@gmail.com">titan.costa@gmail.com</a>> wrote:<br>
>> -    FIXME("(%p/%p)->(%s,%p): stub\n", iface, This, filename, Texture);<br>
>> +    FIXME("iface %p, filename %s, texture %p stub!\n", iface,<br>
>> debugstr_a(filename), texture);<br>
>><br>
><br>
> Do you really need to change the way traces are displayed?<br>
><br>
</div>Yes, you can't use %s for arbitrary application data. The existing<br>
traces are also ugly. Arguably that could have been a separate patch,<br>
but I'm touching the line anyway because the variable name changed.<br>
</blockquote></div><br></div><div class="gmail_extra">No problem with debugstr_a nor variable rename.<br></div><div class="gmail_extra">I meant the format string. The uglyness seems to be a matter of taste.<br></div><div class="gmail_extra">
It is used in many places in wine for COM objects.<br></div><div class="gmail_extra">So unless there is a global will to change traces (like LP stuff removal),<br>that would be better to keep traces as they are for consistency with the rest of the code.<br>
</div></div>