trinity-devel@lists.pearsoncomputing.net

Message: previous - next
Month: September 2010

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

From: "Timothy Pearson" <kb9vqf@...>
Date: Sun, 12 Sep 2010 16:39:09 -0500
> 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.
>
> --- /home/serghei/projects/trinity.old/kdelibs/kdeui/kaction.cpp
> 2010-08-28 16:38:47.000000000 +0300
> +++ /home/serghei/projects/trinity/kdelibs/kdeui/kaction.cpp    2010-09-12
> 17:23:12.000000000 +0300
> @@ -42,6 +42,17 @@
>  #include <ktoolbar.h>
>  #include <ktoolbarbutton.h>
>
> +#include <ft2build.h>
> +#include FT_FREETYPE_H
> +#include <X11/Xdefs.h>
> +#include <X11/Xlib.h>
> +#include <X11/Xatom.h>
> +#include <X11/Intrinsic.h>
> +#include <X11/StringDefs.h>
> +#include <X11/Shell.h>
> +
> +#include <X11/Xft/Xft.h>
> +
>  /**
>  * How it works.
>  * KActionCollection is an organizing container for KActions.
>
> --
> Serghei
>

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