[MPlayer-dev-eng] [PATCH 1/2] Pass application/vnd.ms-opentype attachments to libass
Alexander Strasser
eclipse7 at gmx.net
Sun Oct 26 00:20:12 CEST 2014
On 2014-10-25 13:42 +0200, Nicolas George wrote:
> Le quartidi 4 brumaire, an CCXXIII, Alexander Strasser a écrit :
> > Make those font attachments work with at least demux_mkv.
> >
> > Fix part of ticket 2207
> > Signed-off-by: Alexander Strasser <eclipse7 at gmx.net>
> > ---
> > mplayer.c | 1 +
> > 1 file changed, 1 insertion(+)
>
> I think this is a good idea.
Committed as is because it should fix a reported issue
and I didn't want to delay it.
> > diff --git a/mplayer.c b/mplayer.c
> > index 6a32e88..482bf53 100644
> > --- a/mplayer.c
> > +++ b/mplayer.c
> > @@ -3436,6 +3436,7 @@ goto_enable_cache:
> > if (extract_embedded_fonts &&
> > att->name && att->type && att->data && att->data_size &&
> > (strcmp(att->type, "application/x-truetype-font") == 0 ||
>
> > + strcmp(att->type, "application/vnd.ms-opentype") == 0 ||
>
> This one seems widely used, especially by mkvmerge, so it must be there.
>
> But while you are at it, maybe add the official MIME type for OpenType and
> other fonts, i.e. application/font-sfnt.
I will send a new patch for that.
> > strcmp(att->type, "application/x-font") == 0))
> > ass_add_font(ass_library, att->name, att->data, att->data_size);
> > }
> > --
Thank you for reviewing,
Alexander
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <https://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20141026/46846a32/attachment.asc>
More information about the MPlayer-dev-eng
mailing list