<div dir="ltr"><div dir="ltr">On Fri, Aug 9, 2019 at 10:33 AM Ebrahim Byagowi <<a href="mailto:ebraminio@gmail.com">ebraminio@gmail.com</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Was looking inside libraqm code and saw the use of stdbool.h there, made we wonder why HarfBuzz went for hb_bool_t and whether we can go for it also or not.</div></blockquote><div><br></div><div>Because back in 2006 even stdint.h was not portable-enough.  You still see the following block in hb-common.h:</div><div><br></div><div> #if defined (_SVR4) || defined (SVR4) || defined (__OpenBSD__) || \ </div>    defined (_sgi) || defined (__sun) || defined (sun) || \ <br>    defined (__digital__) || defined (__HP_cc) <br>#  include <inttypes.h> <br>#elif defined (_AIX) <br>#  include <sys/inttypes.h> <br>#elif defined (_MSC_VER) && _MSC_VER < 1600 <br>/* VS 2010 (_MSC_VER 1600) has stdint.h */ <br>typedef __int8 int8_t; <br>typedef unsigned __int8 uint8_t; <br>typedef __int16 int16_t; <br>typedef unsigned __int16 uint16_t; <br>typedef __int32 int32_t; <br>typedef unsigned __int32 uint32_t; <br>typedef __int64 int64_t; <br>typedef unsigned __int64 uint64_t; <br>#else <br>#  include <stdint.h> <br>#endif <br> </div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>If a migration isn't possible, most likely I'd guess, maybe having some compiler definition for using actual bool instead 'int', at least for clients that are embedding harfbuzz can trig some optimizations for them.</div></div></blockquote><div><br></div><div>I don't think it makes any optimizations possible per se.  I'd just forget it...</div><div><br></div><div>We should review if we take any hb_bool_t* arguments in any API.  If not, it's probably safe ABI-wise to change.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><br></div><div>Thanks!</div></div>
_______________________________________________<br>
HarfBuzz mailing list<br>
<a href="mailto:HarfBuzz@lists.freedesktop.org" target="_blank">HarfBuzz@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/harfbuzz" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/harfbuzz</a></blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature">behdad<br><a href="http://behdad.org/" target="_blank">http://behdad.org/</a></div></div>