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

Alex Beregszaszi alex at galaktik.hu
Wed Aug 28 18:21:35 CEST 2002


Hi,

>     Hi!
> 
>     I found some bugs with .rt subtitles. .rt subtitles can have line as:
> 
> <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?

> 
>     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*)

Please remake this part, thx.

-- 
Alex Beregszaszi  <alex at naxine.org>



More information about the MPlayer-dev-eng mailing list