[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-bugs-dist
Subject:    [Bug 96085] Problems with Type 1 as CID fonts
From:       Albert Astals Cid <tsdgeos () terra ! es>
Date:       2005-01-15 13:39:12
Message-ID: 20050115133912.32654.qmail () ktown ! kde ! org
[Download RAW message or body]

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
         
http://bugs.kde.org/show_bug.cgi?id=96085         




------- Additional Comments From tsdgeos terra es  2005-01-15 14:39 -------
CVS commit by aacid: 

Fix my fix of #96085 as freetype treats all CID fonts the same way only from 2.1.8 \
and up Kurt can you check that fixes the problem with characters not beign correctly \
                displayed on firefox nytimes pdf?
CCMAIL: 96085 bugs kde org 
CCMAIL: 95985 bugs kde org 


  M +6 -0      configure.in.in   1.15
  M +9 -4      xpdf/splash/SplashFTFontEngine.cc   1.3


--- kdegraphics/kpdf/configure.in.in  #1.14:1.15
 @ -20,4 +20,10  @
     LIBFREETYPE_CFLAGS="`$FREETYPE_CONFIG --cflags`"
     AC_DEFINE_UNQUOTED(HAVE_FREETYPE, 1, [Defines if your system has the freetype \
library]) +    if test -n "$vers" && test "$vers" -le 9005003; then
+            AC_DEFINE_UNQUOTED(HAVE_FREETYPE_217_OR_OLDER, 1, [Defines if your \
system has the freetype library 2.1.7 or older]) +    else
+            AC_DEFINE_UNQUOTED(HAVE_FREETYPE_217_OR_OLDER, 0, [Defines if your \
system has the freetype library 2.1.7 or older]) +    fi
+            
   else
     AC_MSG_WARN([You need at least libfreetype 2.0.5])

--- kdegraphics/kpdf/xpdf/splash/SplashFTFontEngine.cc  #1.2:1.3
 @ -81,12 +81,17  @ SplashFontFile *SplashFTFontEngine::load
 
   // check for a CFF font
-  /*if ((ff = FoFiType1C::load(fileName))) {
+#if HAVE_FREETYPE_217_OR_OLDER
+  if ((ff = FoFiType1C::load(fileName))) {
     cidToGIDMap = ff->getCIDToGIDMap(&nCIDs);
     delete ff;
-  } else {*/
-  // No need to check for CFF Font, freetype treats all CID fonts the same way
+  } else {
     cidToGIDMap = NULL;
     nCIDs = 0;
-  //}
+  }
+#else
+  // Freetype 2.1.8 and up treats all CID fonts the same way
+  cidToGIDMap = NULL;
+  nCIDs = 0;
+#endif
   ret = SplashFTFontFile::loadCIDFont(this, idA, fileName, deleteFile,
                                       cidToGIDMap, nCIDs);


[prev in list] [next in list] [prev in thread] [next in thread]