Make nil glyph_h_origin() function return true
authorBehdad Esfahbod <behdad@behdad.org>
Thu, 26 Nov 2015 23:43:15 +0000 (18:43 -0500)
committerBehdad Esfahbod <behdad@behdad.org>
Thu, 26 Nov 2015 23:43:15 +0000 (18:43 -0500)
Fixes https://github.com/behdad/harfbuzz/issues/187

Funcs implementations that have a non-zero horizontal origin must
implement the glyph_h_origin() callback, nothing new here.

Other implementations (all I know of!) can simply not set
glyph_h_origin() now.  I did that for hb-ot and hb-ft in
44f82750807475aa5b16099ccccd917d488df703, though that broke the
fallback shaper because the default was returning false...

src/hb-font.cc

index 00f9fb6..d1b8a88 100644 (file)
@@ -109,7 +109,7 @@ hb_font_get_glyph_h_origin_nil (hb_font_t *font HB_UNUSED,
                                void *user_data HB_UNUSED)
 {
   *x = *y = 0;
-  return false;
+  return true;
 }
 static hb_bool_t
 hb_font_get_glyph_h_origin_parent (hb_font_t *font,