[MPlayer-dev-eng] [PATCH] Fix attribute parsing at SRT subtitles
Roberto Togni
rxt at rtogni.it
Mon Jan 18 22:43:48 CET 2016
On Sat, 16 Jan 2016 18:20:26 +0100
Stefan Pöschel <basic.master at gmx.de> wrote:
> I just wanted to ask if there is any further information needed
> regarding the patch.
>
> Am 31.12.2015 um 14:37 schrieb Stefan Pöschel:
> > Hi,
> >
> > when SRT subtitles shall be displayed including formatting, they are
> > converted to ASS markup. The conversion therefore also processes the
> > "color" attribute of the HTML "font" tag.
> >
> > At the moment unfortunately also affects attributes who just have
> > "color" as a prefix, e.g. the attribute "back-color" which is sometimes
> > contained in subtitles. Therefore the value of this background color
> > attribute is by mistake used to set the text color.
> >
> > Please find attached a test SRT file (correct text color: white, not blue).
> >
> > The attached patch fixes this parsing by also including the leading
> > space when recognizing the "color" attribute and also the other attributes.
> >
> > Thanks and regards,
> > Stefan Pöschel
Sorry for the delay.
Applied, thanks.
Ciao,
Roberto
More information about the MPlayer-dev-eng
mailing list