Round, not truncate, when scaling values
authorBehdad Esfahbod <behdad@behdad.org>
Tue, 7 Jun 2016 06:08:54 +0000 (23:08 -0700)
committerBehdad Esfahbod <behdad@behdad.org>
Tue, 7 Jun 2016 06:08:54 +0000 (23:08 -0700)
Fixes https://github.com/behdad/harfbuzz/issues/255

src/hb-font-private.hh

index 8da40a9..0b75577 100644 (file)
@@ -505,7 +505,13 @@ struct hb_font_t {
   }
 
   private:
-  inline hb_position_t em_scale (int16_t v, int scale) { return (hb_position_t) (v * (int64_t) scale / face->get_upem ()); }
+  inline hb_position_t em_scale (int16_t v, int scale)
+  {
+    int upem = face->get_upem ();
+    int64_t scaled = v * (int64_t) scale;
+    scaled += scaled >= 0 ? upem/2 : -upem/2; /* Round. */
+    return (hb_position_t) (scaled / upem);
+  }
 };
 
 #define HB_SHAPER_DATA_CREATE_FUNC_EXTRA_ARGS