[MPlayer-dev-eng] A bit more liberal parsing of tags in .srt files?

[Szokovacs Robert]

On 2012 Mar 03, Sat 15:04:35 Clément Bœsch wrote:
> On Sat, Mar 03, 2012 at 02:57:18PM +0100, Reimar Döffinger wrote:
> > On Wed, Feb 22, 2012 at 09:50:42PM +0100, SZOKOVACS Robert wrote:
> > > On 2012. February 16. Thu 07:39:55 compn wrote:
> > > > On Thu, 16 Feb 2012 11:13:01 +0100, Szokovacs Robert wrote:
> > > > >Hi,
> > > > >
> > > > >A lot of .srt subtitle files I encounter on the net is deemed
> > > > >incorrect by
> > > > >the current mplayer, so the <font> tag is displayed as if it was part
> > > > >of
> > > > >the script.
> > > > >(example: http://www.tvsubtitles.net/download-204587.html, some
> > > > >color=
> > > > 
> > > > i think we shouldnt render any <tag> with < and >.
> > > > but instead silently drop it.
> > > > 
> > > > i dont know whats best re: your idea. better wait for reimar.
> > > 
> > > Reimar, would you please give us some pointers?
> > 
> > I guess looking for < followed by a-z and where a > exists before the
> > next < is about as reliable as you can make it if you want it to
> > catch every case (well, the tags itself, handling hypothetical stuff
> > like <comment>everything here should not be shown</comment> is
> > definitely out of scope for MPlayer IMO.
> 
> Not dropping the invalid tags was intentional; it helps showing "invalid"
> samples. IMO it would be wise to instead fix the code to support
> attributes specified without quotes.

Hi,

Attached is the patch doing this change.
It also contains the color aliases mentioned it Federico Kereki's email at 
12/03/24

br

Szo
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mplayersub.patch
Type: text/x-patch
Size: 6035 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20120413/4ae9f596/attachment.bin>