Fixes #322174:

2005-11-22  Federico Mena Quintero  <federico@ximian.com>

        Fixes #322174:

        * pango/pangofc-font.h (struct _PangoFcFont): Replace the
        "gpointer context_key" field with "gpointer priv".  This way we
        can access the private data quickly, instead of using
        g_type_instance_get_private().

        * pango/pangofc-private.h: Added prototypes for
        _pango_fc_font_{get,set}_context_key().

        * pango/pangofc-font.c (struct _PangoFcFontPrivate): Moved the
        "context_key" field to here.
        (PANGO_FC_FONT_GET_PRIVATE): Use the "priv" field instead of GType
        private data.
        (pango_fc_font_class_init): Don't register GType private data.
        (pango_fc_font_init): Initialize the private data here.
        (pango_fc_font_finalize): Free the private data.
        (_pango_fc_font_get_context_key): Implement.
        (_pango_fc_font_set_context_key): Implement.
        (pango_fc_font_get_glyph): Remove the g_return_val_if_fail(); it
        was appearing quite high in the profile.

        * pango/pangofc-fontmap.c (pango_fc_font_map_add): Call
        _pango_fc_font_set_context_key() instead of setting the
        fcfont->context_key directly.
        (_pango_fc_font_map_remove): Likewise; also use
        _pango_fc_font_get_context_key() instead of accessing the field
        directly.

1 files changed, 25 insertions, 6 deletions

diff --git a/pango/pangofc-font.c b/pango/pangofc-font.c
index 882e77e4..fe01a5b7 100644
--- a/pango/pangofc-font.c
+++ b/pango/pangofc-font.c
@@ -37,11 +37,12 @@ enum {
 
 typedef struct _PangoFcFontPrivate PangoFcFontPrivate;
 
-#define PANGO_FC_FONT_GET_PRIVATE(obj) (G_TYPE_INSTANCE_GET_PRIVATE ((obj), PANGO_TYPE_FC_FONT, PangoFcFontPrivate))
+#define PANGO_FC_FONT_GET_PRIVATE(obj) ((PangoFcFontPrivate *) ((PangoFcFont *) obj)->priv)
 
 struct _PangoFcFontPrivate
 {
   PangoFcDecoder *decoder;
+  gpointer context_key;
 };
 
 static gboolean pango_fc_font_real_has_char  (PangoFcFont *font,
@@ -95,13 +96,15 @@ pango_fc_font_class_init (PangoFcFontClass *class)
 							 "Pattern",
 							 "The fontconfig pattern for this font",
 							 G_PARAM_WRITABLE | G_PARAM_CONSTRUCT_ONLY));
-
-  g_type_class_add_private (object_class, sizeof (PangoFcFontPrivate));
 }
 
 static void
 pango_fc_font_init (PangoFcFont *fcfont)
 {
+  PangoFcFontPrivate *priv;
+
+  priv = g_new0 (PangoFcFontPrivate, 1);
+  fcfont->priv = priv;
 }
 
 static void
@@ -129,6 +132,8 @@ pango_fc_font_finalize (GObject *object)
   if (priv->decoder)
     _pango_fc_font_set_decoder (fcfont, NULL);
 
+  g_free (priv);
+
   G_OBJECT_CLASS (pango_fc_font_parent_class)->finalize (object);
 }
 
@@ -606,11 +611,8 @@ pango_fc_font_get_glyph (PangoFcFont *font,
 {
   PangoFcFontPrivate *priv;
   
-  g_return_val_if_fail (PANGO_IS_FC_FONT (font), 0);
-
   priv = PANGO_FC_FONT_GET_PRIVATE (font);
 
-
   /* Replace NBSP with a normal space; it should be invariant that
    * they shape the same other than breaking properties.
    */
@@ -746,6 +748,23 @@ _pango_fc_font_set_decoder (PangoFcFont    *font,
     g_object_ref (priv->decoder);
 }
 
+gpointer
+_pango_fc_font_get_context_key (PangoFcFont *fcfont)
+{
+  PangoFcFontPrivate *priv = PANGO_FC_FONT_GET_PRIVATE (fcfont);
+
+  return priv->context_key;
+}
+
+void
+_pango_fc_font_set_context_key (PangoFcFont *fcfont,
+				gpointer     context_key)
+{
+  PangoFcFontPrivate *priv = PANGO_FC_FONT_GET_PRIVATE (fcfont);
+
+  priv->context_key = context_key;
+}
+
 static FT_Glyph_Metrics *
 get_per_char (FT_Face      face,
 	      FT_Int32     load_flags,