[HarfBuzz] HarfBuzz code in CVS
behdad at cs.toronto.edu
Thu Apr 6 12:46:22 PDT 2006
On Thu, 6 Apr 2006, Lars Knoll wrote:
> On Thursday 06 April 2006 20:14, Behdad Esfahbod wrote:
> > On Thu, 6 Apr 2006, it was written:
> > > On Thu, 6 Apr 2006, Lars Knoll wrote:
> > > > Done.
> > >
> > > Thanks. I'll review them. Most of the warnings as I remember
> > > were simply signedness issues.
> > Ok, I see you fixed the unused-parameter ones too. Don't know if
> > FT_UNUSED has any runtime penalty or not, but it's definitely
> > cleaner and easier to setup than using the gcc attributes. I
> > just added some bits to harfbuzz-impl.h to define it if it's not
> > defined, just in case it's not in older FreeType versions...
> Good. I don't think it has runtime issues. At least gcc-4 should be able to
> optimise that code out. But that's easy to check :)
Sure it is. Glad gcc doesn't detect that and issue the warning
> And I'll remove the dumper part from harfbuzz.c to reduce the size of the
> embedded code tomorrow.
A neat way to do that is to add .h-style #ifdef's around the C
files and just pass -DHARFBUZZ_DUMP_C in your Makefile, so you
don't have to patch the files.
"Commandment Three says Do Not Kill, Amendment Two says Blood Will Spill"
-- Dan Bern, "New American Language"
More information about the HarfBuzz