[HarfBuzz] harfbuzz: Branch 'master' - 3 commits

Behdad Esfahbod behdad at kemper.freedesktop.org
Wed Mar 6 17:07:22 PST 2013


 configure.ac                     |    1 +
 src/hb-ft.cc                     |    4 ++--
 src/hb-ot-shape-complex-indic.cc |    2 +-
 3 files changed, 4 insertions(+), 3 deletions(-)

New commits:
commit a8ac1d33c731f6200ee472ab6c37a80b72770564
Author: Behdad Esfahbod <behdad at behdad.org>
Date:   Wed Mar 6 20:05:34 2013 -0500

    [build] Add AM_PROG_AR
    
    Newer automake / libtools warns otherwise.

diff --git a/configure.ac b/configure.ac
index b4249cd..061880f 100644
--- a/configure.ac
+++ b/configure.ac
@@ -13,6 +13,7 @@ AM_INIT_AUTOMAKE([1.11.1 gnits dist-bzip2 no-dist-gzip -Wall no-define color-tes
 AM_SILENT_RULES([yes])
 
 # Initialize libtool
+AM_PROG_AR
 LT_PREREQ([2.2])
 LT_INIT([disable-static])
 
commit 5594c2d112c295147ee69215a2ce9dfd99984aa0
Author: Behdad Esfahbod <behdad at behdad.org>
Date:   Wed Mar 6 19:37:31 2013 -0500

    [FT] Just return if glyph name not found
    
    The fallback happens in higher level already.  No need to do here.

diff --git a/src/hb-ft.cc b/src/hb-ft.cc
index 1e87c1c..a25d5a4 100644
--- a/src/hb-ft.cc
+++ b/src/hb-ft.cc
@@ -242,8 +242,8 @@ hb_ft_get_glyph_name (hb_font_t *font HB_UNUSED,
   FT_Face ft_face = (FT_Face) font_data;
 
   hb_bool_t ret = !FT_Get_Glyph_Name (ft_face, glyph, name, size);
-  if (!ret || (size && !*name))
-    snprintf (name, size, "gid%u", glyph);
+  if (ret && (size && !*name))
+    ret = false;
 
   return ret;
 }
commit fb7c182bf92142540bff1ad7fb82de0d115fb2b5
Author: Behdad Esfahbod <behdad at behdad.org>
Date:   Wed Mar 6 00:53:24 2013 -0500

    [Indic] Minor

diff --git a/src/hb-ot-shape-complex-indic.cc b/src/hb-ot-shape-complex-indic.cc
index 50d73e6..ddb1839 100644
--- a/src/hb-ot-shape-complex-indic.cc
+++ b/src/hb-ot-shape-complex-indic.cc
@@ -733,7 +733,7 @@ initial_reordering_consonant_syllable (const hb_ot_shape_plan_t *plan,
 	     *
 	     * IMPLEMENTATION NOTES:
 	     *
-	     * Our pre-base reordering Ra's are marked POS_BELOW, so will be skipped
+	     * Our pre-base reordering Ra's are marked POS_POST_C, so will be skipped
 	     * by the logic above already.
 	     */
 



More information about the HarfBuzz mailing list