trinity-devel@lists.pearsoncomputing.net

Message: previous - next
Month: September 2010

Re: [trinity-devel] #include FT_FREETYPE_H in kdelibs/kdeui/kaction.cpp

From: Serghei Amelian <serghei@...>
Date: Mon, 13 Sep 2010 00:55:55 +0300
On Monday 13 September 2010 00:39:09 Timothy Pearson wrote:
> > Hello,
> >
> > I found that new revision of kdelibs contains these new headers. Are
> > really
> > needed there? In any case, i cannot found where FT_FREETYPE_H is set.

[...]

> I don't know why "#include FREETYPE" is present, but the X11 includes are
> required.  I would personally leave FT_FREETYPE_H in place unless it is
> causing a build failure under CMake, in which case it can be removed as we
> are not yet in the feature freeze.
>
> Tim

Yes, build failing because FT_FREETYPE_H is not defined. Also, compiling fine 
even without X11/freetype inclusions. I can't see in source any X11/freetype 
calls.

Actually 3 files have this strange inclusion: kactionclasses.cpp, kaction.cpp, 
and kfontcombo.cpp.

-- 
Serghei