[HarfBuzz] HarfBuzz code in CVS

[Lars Knoll]

On Thursday 06 April 2006 10:13, Lars Knoll wrote:
> On Sunday 02 April 2006 10:19, Lars Knoll wrote:
> > On Saturday 01 April 2006 04:24, Behdad Esfahbod wrote:
> > > On Fri, 31 Mar 2006, Lars Knoll wrote:
> > > > On Friday 31 March 2006 15:15, Behdad Esfahbod wrote:
> > > > > I spent last night restructuring and namespacing the OpenType
> > > > > Layout code in Pango, and the result is in HarfBuzz CVS in the
> > > > > new module 'harfbuzz'.
> > > >
> > > > Great. Thanks for doing that work. I'll update our codebase in Qt and
> > > > test it next week (it's almost weekend here now :)
> > >
> > > No problem.  It was something I wanted to do for quite a while.
> > > I like how the final codebase came out, and it was a night worth
> > > of work only...
> >
> > Great :) Can't wait to look at it.
>
> I've looked at it now. It took me about 5 minutes to convert Qt over to
> using the changed code :)
>
> I've made the adjustments in Qt, and from Qt 4.2 onwards we'll be using a
> simple copy of the harfbuzz CVS repository.
>
> > > > > Lars, can you test?  For your convenience, you can simply compile
> > > > > harfbuzz.c that includes all the source files.
>
> Only problem I could see is that the code generates quite some compiler
> warnings. I'll fix them in the next hours.

Done. 

While fixing the warnings I actually noticed one thing: harfbuzz.c includes 
harfbuzz-dump.c. I would like to remove that include, as it's only used to 
debug harfbuzz (or Qt/pango's opentype code), but not included in production 
code.

Cheers,
Lars