[MPlayer-dev-eng] [PATCH] two pathes: RT subtitles and info fields for MEncoder

[Arpi]

Hi,

> > <Time Begin="4.1" End="7.5"><clear/>Some text
> > <Time Begin="1:4:52.7"><clear/>Other text
> > 
> > First patch fixes it. It works fine now with all subtitles I have.
> This looks ok, should I commit it?
i found it messy but i had no time to understand the patch, so maybe it's ok

> >     Second patch implements some avi info fields into MEncoder. These
> > fields are: Software (MEncoder, of course :), Name, Artist, Genre,
> > Subject, Copyright, Source, Comment. It also fixes broken RIFF header
> > (I wonder why nobody noticed that yet) in generated avi file.
> This isn't the right way to do it. Do not make so many non-sense variables!
> (char *info_* and int infotype*)
agree. and it must be a suboption thing, like -info copyright=...,title=...


A'rpi / Astral & ESP-team

--
Developer of MPlayer, the Movie Player for Linux - http://www.MPlayerHQ.hu