<br><div class="gmail_quote">2013/4/30 John Emmas <span dir="ltr"><<a href="mailto:johne53@tiscali.co.uk" target="_blank">johne53@tiscali.co.uk</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 30/04/2013 07:41, Konstantin Ritt wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Behdad,<br>
<br>
John is probably right, having DEBUG macro defined in release mode could potentially lead to a hard-tracking issues. And this possibility grows when we're compiling Harfbuzz in instead of linking to it.<br>
Maybe worth of renaming to HB_DEBUG?<br>
<br>
</blockquote>
<br></div>
Thanks for supporting me on this, Konstantin.<br>
<br>
In defence of Harfbuzz I should emphasize that at present, there's absolutely no problem with the custom macro.  It gets defined in 'hb-private.hh' which is never exposed through any of the public header files.  However, my fear is that this is the kind of thing which tends to get get forgotten over time.  In the future, somebody could take the macro out of 'hb-private.hh' and innocuously move it to some other header file - where it could end up causing havoc.  I totally agree with renaming it to HB_DEBUG.  Otherwise, there's effectively an accident waiting to happen.</blockquote>

<div><br></div><div>True. But in case Harfbuzz gets compiled-in, DEBUG might be defined outside of hb-private.hh, leading to HB build failure.</div><div><br></div><div>
Kind regards,<br>Konstantin
</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><br>
<br>
John<br>
_______________________________________________<br>
HarfBuzz mailing list<br>
<a href="mailto:HarfBuzz@lists.freedesktop.org" target="_blank">HarfBuzz@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/harfbuzz" target="_blank">http://lists.freedesktop.org/mailman/listinfo/harfbuzz</a><br>
</div></div></blockquote></div><br>