Changeset 704 for trunk/include/mapnik/font_engine_freetype.hpp

Timestamp:

06/29/08 06:58:48 (6 months ago)

Author:

artem

Message:

applied font fallback patch from Beau Gunderson

Files:

• trunk/include/mapnik/font_engine_freetype.hpp (modified) (12 diffs)

trunk/include/mapnik/font_engine_freetype.hpp

@@ -31 +31 @@
 #include <mapnik/geometry.hpp>
 #include <mapnik/text_path.hpp>
+#include <mapnik/font_set.hpp>
 
 // freetype2
@@ -62 +63 @@
     {
     public:
-        typedef std::pair<unsigned,unsigned> dimension_t;
-
-        font_face(FT_Face face)
+        font_face(FT_Face face)
             : face_(face) {}
         
@@ -105 +104 @@
         }
               
+        ~font_face()
+        {
+#ifdef MAPNIK_DEBUG
+            std::clog << "~font_face: Clean up face \"" << family_name()
+                << " " << style_name() << "\"" << std::endl;
+#endif
+            FT_Done_Face(face_);
+        }
+        
+    private:
+        FT_Face face_;
+    };
+    
+    typedef boost::shared_ptr<font_face> face_ptr;
+   
+    class MAPNIK_DECL freetype_engine  // : public mapnik::singleton<freetype_engine,mapnik::CreateStatic>,
+         // private boost::noncopyable
+    {
+       // friend class mapnik::CreateStatic<freetype_engine>;
+      public:
+        static bool register_font(std::string const& file_name);
+        static std::vector<std::string> face_names ();
+        face_ptr create_face(std::string const& family_name);
+        virtual ~freetype_engine();
+        freetype_engine();
+      private:
+        FT_Library library_;
+        static boost::mutex mutex_;
+        static std::map<std::string,std::string> name2file_;
+    }; 
+    
+    template <typename T>
+    class MAPNIK_DECL face_manager : private boost::noncopyable
+    {
+        typedef T font_engine_type;
+        typedef std::map<std::string,face_ptr> faces;
+        
+    public:
+        face_manager(T & engine)
+           : engine_(engine) {}
+        
+        face_ptr get_face(std::string const& name)
+        {
+            typename faces::iterator itr;
+            itr = faces_.find(name);
+            if (itr != faces_.end())
+            {
+                return itr->second;
+            }
+            else
+            {
+                face_ptr face = engine_.create_face(name);
+                if (face)
+                {
+                    faces_.insert(make_pair(name,face));
+                }
+                return face;    
+            }
+        }
+    private:
+        faces faces_;
+        font_engine_type & engine_;
+    };
+        
+    template <typename T>
+    struct text_renderer : private boost::noncopyable
+    {
+        typedef std::pair<unsigned,unsigned> dimension_t;
+        
+        struct glyph_t : boost::noncopyable
+        {
+            FT_Glyph image;
+            glyph_t(FT_Glyph image_) : image(image_) {}
+            ~glyph_t () { FT_Done_Glyph(image);}
+        };
+        
+        typedef boost::ptr_vector<glyph_t> glyphs_t;
+        typedef T pixmap_type;
+        
+        text_renderer (pixmap_type & pixmap, std::vector<face_ptr> faces)
+            : pixmap_(pixmap),
+              faces_(faces),
+              fill_(0,0,0), 
+              halo_fill_(255,255,255),
+              halo_radius_(0) {}
+
         dimension_t character_dimensions(const unsigned c)
         {
@@ -111 +196 @@
             FT_Error  error;
             
-            FT_GlyphSlot slot = face_->glyph;
-            
             pen.x = 0;
             pen.y = 0;
@@ -118 +201 @@
             FT_BBox glyph_bbox; 
             FT_Glyph image;
-            
+                    
+            FT_Face face = (*faces_.begin())->get_face();
+            
+            FT_UInt glyph_index = FT_Get_Char_Index(face, c);
+                 
+            // If there's no glyph_index we loop through the remaining fonts
+            // in the fontset looking for one.
+            if (!glyph_index) {
+                std::vector<face_ptr>::iterator itr = faces_.begin();
+                std::vector<face_ptr>::iterator end = faces_.end();
+                
+                ++itr; // Skip the first one, we already tried it.
+                
+                for (; itr != end; ++itr)
+                {
+                    FT_Face f = (*itr)->get_face();
+                    
+                    glyph_index = FT_Get_Char_Index(f, c);
+
+                    if (glyph_index) {
+                        face = f;
+                        break;
+                    }
+                }
+            }
+
             matrix.xx = (FT_Fixed)( 1 * 0x10000L ); 
             matrix.xy = (FT_Fixed)( 0 * 0x10000L ); 
             matrix.yx = (FT_Fixed)( 0 * 0x10000L ); 
             matrix.yy = (FT_Fixed)( 1 * 0x10000L );
-                    
-            FT_Set_Transform (face_,&matrix,&pen);
-            
-            FT_UInt glyph_index = FT_Get_Char_Index( face_, c);
-            
-            error = FT_Load_Glyph (face_,glyph_index,FT_LOAD_NO_HINTING); 
+            
+            FT_Set_Transform(face, &matrix, &pen);
+
+            error = FT_Load_Glyph (face, glyph_index, FT_LOAD_NO_HINTING); 
             if ( error )
                 return dimension_t(0, 0);
             
-            error = FT_Get_Glyph( face_->glyph, &image);
+            error = FT_Get_Glyph(face->glyph, &image);
             if ( error )
                 return dimension_t(0, 0);
-            
-            FT_Glyph_Get_CBox(image,ft_glyph_bbox_pixels, &glyph_bbox); 
-            FT_Done_Glyph(image); 
-            return dimension_t(slot->advance.x >> 6, glyph_bbox.yMax - glyph_bbox.yMin);
+
+            FT_Glyph_Get_CBox(image, ft_glyph_bbox_pixels, &glyph_bbox); 
+            FT_Done_Glyph(image);
+
+            unsigned tempx = face->glyph->advance.x >> 6;
+            unsigned tempy = glyph_bbox.yMax - glyph_bbox.yMin;
+
+            //std::clog << "glyph: " << glyph_index << " x: " << tempx << " y: " << tempy << std::endl;
+
+            return dimension_t(tempx, tempy);
         }
         
@@ -188 +300 @@
                         if ( *arabic.getBuffer() >= 0x0600 && *arabic.getBuffer() <= 0x06ff)
                         {
-                            
                            UnicodeString shaped;
                            u_shapeArabic(arabic.getBuffer(),arabic.length(),shaped.getBuffer(arabic.length()),arabic.length(),
@@ -216 +327 @@
             info.set_dimensions(width, height);
         }
-        
-        ~font_face()
-        {
-#ifdef MAPNIK_DEBUG
-            std::clog << "clean up face:" << family_name()<<":" << style_name() << std::endl;
-#endif
-            FT_Done_Face(face_);
-        }
-        
-    private:
-        FT_Face face_;
-    };
-    
-   typedef boost::shared_ptr<font_face> face_ptr;
-   class MAPNIK_DECL freetype_engine  // : public mapnik::singleton<freetype_engine,mapnik::CreateStatic>,
-         // private boost::noncopyable
-    {
-       // friend class mapnik::CreateStatic<freetype_engine>;
-      public:
-        static bool register_font(std::string const& file_name);
-        static std::vector<std::string> face_names ();
-        face_ptr create_face(std::string const& family_name);
-        virtual ~freetype_engine();
-        freetype_engine();
-      private:
-        FT_Library library_;
-        static boost::mutex mutex_;
-        static std::map<std::string,std::string> name2file_;
-    }; 
-    
-    template <typename T>
-    class MAPNIK_DECL face_manager : private boost::noncopyable
-    {
-        typedef T font_engine_type;
-        typedef std::map<std::string,face_ptr> faces;
-        
-    public:
-        face_manager(T & engine)
-           : engine_(engine) {}
-        
-        face_ptr get_face(std::string const& name)
-        {
-            typename faces::iterator itr;
-            itr = faces_.find(name);
-            if (itr != faces_.end())
-            {
-                return itr->second;
-            }
-            else
-            {
-                face_ptr face = engine_.create_face(name);
-                if (face)
-                {
-                    faces_.insert(make_pair(name,face));
-                }
-                return face;    
-            }
-        }
-    private:
-        faces faces_;
-        font_engine_type & engine_;
-    };
-        
-    template <typename T>
-    struct text_renderer : private boost::noncopyable
-    {
-        struct glyph_t : boost::noncopyable
-        {
-            FT_Glyph image;
-            glyph_t(FT_Glyph image_) : image(image_) {}
-            ~glyph_t () { FT_Done_Glyph(image);}
-        };
-        
-        typedef boost::ptr_vector<glyph_t> glyphs_t;
-        typedef T pixmap_type;
-        
-        text_renderer (pixmap_type & pixmap, face_ptr face)
-            : pixmap_(pixmap),
-              face_(face),
-              fill_(0,0,0), 
-              halo_fill_(255,255,255),
-              halo_radius_(0) {}
-    
+
         void set_pixel_size(unsigned size)
         {
-            face_->set_pixel_sizes(size);
+            std::vector<face_ptr>::iterator itr = faces_.begin();
+            std::vector<face_ptr>::iterator end = faces_.end();
+            for (; itr != end; ++itr)
+            {
+                (*itr)->set_pixel_sizes(size);
+            }
         }
     
@@ -327 +361 @@
             FT_Vector pen;
             FT_Error  error;
-            
-            FT_Face face = face_->get_face();
-            //            FT_GlyphSlot slot = face->glyph;
-            
+
             FT_BBox bbox;   
-            bbox.xMin = bbox.yMin = 32000; 
-            bbox.xMax = bbox.yMax = -32000; //hmm?? 
-            
+            bbox.xMin = bbox.yMin = 32000;  // Initialize these so we can tell if we
+            bbox.xMax = bbox.yMax = -32000; // properly grew the bbox later
+            
+            std::vector<face_ptr>::iterator end = faces_.end();
+            
             for (int i = 0; i < path->num_nodes(); i++) 
             {
                 int c;
                 double x, y, angle;
-                
+
                 path->vertex(&c, &x, &y, &angle);
-//                std::clog << "   prepare_glyph: " << (unsigned char)c << "," << x << "," << y << "," << angle << std::endl;
-
+
+#ifdef MAPNIK_DEBUG
+                // TODO Enable when we have support for setting verbosity
+                //std::clog << "prepare_glyphs: " << c << "," << x << 
+                //    "," << y << "," << angle << std::endl;
+#endif
 
                 FT_BBox glyph_bbox; 
@@ -350 +387 @@
                 pen.y = int(y * 64);
                         
+                FT_Face face = (*faces_.begin())->get_face();
+
+                FT_UInt glyph_index = FT_Get_Char_Index(face, unsigned(c));
+                     
+                // If there's no glyph_index we loop through the remaining fonts
+                // in the fontset looking for one.
+                if (!glyph_index) {
+                    std::vector<face_ptr>::iterator itr = faces_.begin();
+                    ++itr; // Skip the first one, we already tried it.
+                    
+                    for (; itr != end; ++itr)
+                    {
+#ifdef MAPNIK_DEBUG
+                        // TODO Enable when we have support for setting verbosity
+                        //std::clog << "prepare_glyphs: Falling back to font named \""
+                        //    << (*itr)->family_name() << " " << (*itr)->style_name()
+                        //    << "\"" << std::endl;
+#endif
+
+                        FT_Face f = (*itr)->get_face();
+                        
+                        glyph_index = FT_Get_Char_Index(f, unsigned(c));
+
+                        if (glyph_index) {
+                            face = f;
+                            break;
+                        }
+                    }
+
+#ifdef MAPNIK_DEBUG
+                    // TODO Enable when we have support for setting verbosity
+                    //if (!glyph_index) {
+                    //    std::clog << "prepare_glyphs: Failed to fall back, glyph "
+                    //        << c << " not found in any font." << std::endl;
+                    //}
+#endif
+                }
+
                 matrix.xx = (FT_Fixed)( cos( angle ) * 0x10000L ); 
                 matrix.xy = (FT_Fixed)(-sin( angle ) * 0x10000L ); 
                 matrix.yx = (FT_Fixed)( sin( angle ) * 0x10000L ); 
                 matrix.yy = (FT_Fixed)( cos( angle ) * 0x10000L );
-                
-                FT_Set_Transform (face,&matrix,&pen);
-                
-                FT_UInt glyph_index = FT_Get_Char_Index( face, unsigned(c));
-                
-                error = FT_Load_Glyph (face,glyph_index, FT_LOAD_NO_HINTING); 
+
+                FT_Set_Transform(face, &matrix, &pen);
+
+                error = FT_Load_Glyph(face, glyph_index, FT_LOAD_NO_HINTING); 
                 if ( error )
                     continue;
-                
-                error = FT_Get_Glyph( face->glyph, &image);
+
+                error = FT_Get_Glyph(face->glyph, &image);
                 if ( error )
                     continue;
-                
+
                 FT_Glyph_Get_CBox(image,ft_glyph_bbox_pixels, &glyph_bbox); 
                 if (glyph_bbox.xMin < bbox.xMin) 
@@ -376 +449 @@
                 if (glyph_bbox.yMax > bbox.yMax) 
                     bbox.yMax = glyph_bbox.yMax;
-                
+        
+                // Check if we properly grew the bbox
                 if ( bbox.xMin > bbox.xMax )
                 {
@@ -485 +559 @@
     
         pixmap_type & pixmap_;
-        face_ptr face_;
+        std::vector<face_ptr> faces_;
         mapnik::Color fill_;
         mapnik::Color halo_fill_;
@@ -495 +569 @@
 }
 
-
 #endif // FONT_ENGINE_FREETYPE_HPP