]> granicus.if.org Git - libass/commitdiff
fontselect: improve font display name choice
authorGrigori Goronzy <greg@chown.ath.cx>
Mon, 7 Sep 2015 09:03:28 +0000 (11:03 +0200)
committerGrigori Goronzy <greg@chown.ath.cx>
Mon, 7 Sep 2015 09:03:28 +0000 (11:03 +0200)
libass/ass_fontselect.c

index a71ad4ef58ed3ba6e50a3cd3718eb455adb037d6..2be65497a3a17ba52c09944929ff198ab9b769c6 100644 (file)
@@ -560,9 +560,13 @@ find_font(ASS_FontSelector *priv, ASS_Library *library,
             ASS_FontProvider *provider = selected->provider;
             stream->func = provider->funcs.get_data;
             stream->priv = selected->priv;
-            // FIXME: we should define a default family name in some way,
-            // possibly the first (or last) English name
-            result = strdup(selected->families[0]);
+            // Prefer PostScript name because it is unique. This is only
+            // used for display purposes so it doesn't matter that much,
+            // though.
+            if (selected->postscript_name)
+                result = strdup(selected->postscript_name);
+            else
+                result = strdup(selected->families[0]);
         } else
             result = strdup(selected->path);
     }