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

[Clément Bœsch]

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.

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20120303/7ab5686a/attachment.asc>