Adjust italic to non-italic style spacing

Previously, the space added was quite a lot, since it was calculated so
that it would be enough to accommodate a heavily FreeType-slanted glyph
in the worst case. In many cases this was too much spacing, especially
if the italic font was only slightly angled.

The new fix simply ensures that the glyph is inside the bounds of the
advance.

diff --git a/libass/ass_render.c b/libass/ass_render.c
index 8d55667018db8c6abc8293b75116e550d3de0246..719b8bdbdeca918d75abaf73d2939abb47da0dec 100644 (file)
--- a/libass/ass_render.c
+++ b/libass/ass_render.c
@@ -1859,10 +1859,8 @@ ass_render_event(ASS_Renderer *render_priv, ASS_Event *event,
             while (back && og->bbox.xMax - og->bbox.xMin == 0
                     && og->italic)
                 og = &glyphs[--back];
-            if (og->bbox.xMax > og->cluster_advance.x) {
-                // The FreeType oblique slants by 6/16
-                og->cluster_advance.x += og->bbox.yMax * 0.375;
-            }
+            if (og->bbox.xMax > og->cluster_advance.x)
+                og->cluster_advance.x = og->bbox.xMax;
         }
 
         // add horizontal letter spacing