From 67a4ca003729aad821a097a84cfd11b7c16228ea Mon Sep 17 00:00:00 2001 From: Oleg Oshmyan Date: Sat, 17 Oct 2020 21:40:27 +0300 Subject: Ignore font substitution when splitting combined bitmaps Well, that was easy. font->desc.family is the raw family name from FontName/\fn, barring the leading @ for vertical fonts. So, for reference, even if the script requests two different font names and neither exists on the system and font fallback picks the same font for both, is_new_bm_run will treat them as distinct. This is indeed the desired behaviour that matches VSFilter. --- libass/ass_render.c | 2 -- 1 file changed, 2 deletions(-) (limited to 'libass') diff --git a/libass/ass_render.c b/libass/ass_render.c index b00f22a..ce9503a 100644 --- a/libass/ass_render.c +++ b/libass/ass_render.c @@ -1858,11 +1858,9 @@ fix_glyph_scaling(ASS_Renderer *priv, GlyphInfo *glyph) */ static bool is_new_bm_run(GlyphInfo *info, GlyphInfo *last) { - // FIXME: Don't break on glyph substitutions return !last || info->effect || info->drawing_text || last->drawing_text || strcmp(last->font->desc.family, info->font->desc.family) || last->font->desc.vertical != info->font->desc.vertical || - last->face_index != info->face_index || last->font_size != info->font_size || last->c[0] != info->c[0] || last->c[1] != info->c[1] || -- cgit v1.2.3