Thanks for the quick fix!<div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Sep 10, 2012 at 11:00 PM, Tom Hacohen <span dir="ltr"><<a href="mailto:tom.hacohen@samsung.com" target="_blank">tom.hacohen@samsung.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On 10/09/12 16:57, Behdad Esfahbod wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 09/10/2012 09:51 AM, Kenichi Ishibashi wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Behdad,<br>
<br>
The current revision can't compile with clang++ if -Werror option is specified.<br>
<br>
src/hb-open-type-private.hh:<u></u>437:93: error: initialization of pointer of type<br>
'OT::GenericArrayOf<OT::<u></u>IntType<unsigned short>, OT::OffsetTo<OT::LigatureSet><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
*' to null from a constant boolean expression [-Werror,-Wbool-conversion]<br>
</blockquote>
     if (unlikely (!this->allocate_size<Type> (((char *) &obj) + size -<br>
this->head))) return false<br>
<br>
I'd appreciate if you could use NULL instead of false here.<br>
</blockquote>
<br>
Oops!  Fixed.  Thanks.<br>
<br>
I wish gcc was more helpful.  One day I will switch to clang.  One day...<br>
</blockquote>
<br></div></div>
At least consider compiling with the clang static analyser so you'll get all the errors and warnings clang usually provides with the added bonus of static analysing. :)<br>
<br>
--<br>
Tom.<br>
<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>Kenichi Ishibashi<div><a href="mailto:bashi@google.com" target="_blank">bashi@google.com</a></div><br>
</div>