Add a comment about using cairo_scaled_font_text_extents instead of

2006-01-20  Behdad Esfahbod  <behdad@gnome.org>

        * pango/pangocairo-font.c (_pango_cairo_get_hex_box_info): Add a
        comment about using cairo_scaled_font_text_extents instead of
        cairo_text_extents.

2 files changed, 12 insertions, 1 deletions

diff --git a/ChangeLog b/ChangeLog
index c131aad4..687ee089 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2006-01-20  Behdad Esfahbod  <behdad@gnome.org>
+
+	* pango/pangocairo-font.c (_pango_cairo_get_hex_box_info): Add a
+	comment about using cairo_scaled_font_text_extents instead of
+	cairo_text_extents.
+
 2006-01-16  Behdad Esfahbod  <behdad@gnome.org>
 
 	* === Released 1.11.2 ===
diff --git a/pango/pangocairo-font.c b/pango/pangocairo-font.c
index 37c7834f..a727e053 100644
--- a/pango/pangocairo-font.c
+++ b/pango/pangocairo-font.c
@@ -171,7 +171,12 @@ _pango_cairo_get_hex_box_info (PangoCairoFont *cfont)
       cairo_text_extents_t extents;
 
       c[0] = hexdigits[i];
-      cairo_text_extents (cr, c, &extents);      
+      /* The following call is being added into cairo.  That's preferred way
+       * to get the extents here.  When turning that on, we can get rid of
+       * the dummy surface and cairo context created above.
+       */
+      /* cairo_scaled_font_text_extents (scaled_mini_font, c, &extents); */
+      cairo_text_extents (cr, c, &extents);
       width = MAX (width, extents.width);
       height = MAX (height, extents.height);
     }