[FFmpeg-cvslog] r15211 - trunk/libavformat/matroskadec.c

[Aurelien Jacobs]

Joakim Plate wrote:

> aurel <subversion <at> mplayerhq.hu> writes:
> 
> > 
> > Author: aurel
> > Date: Fri Sep  5 01:26:12 2008
> > New Revision: 15211
> > 
> > Log:
> > matroskadec: fix ASS subtitle track packets before emitting them
> > Matroska does some butchering when storing the ASS lines. The start and end
> > time are removed (because they are duplicated in the container).
> > The matroska_fix_ass_packet() function simply restore those start and end
> > time in ASS lines to ensure our ASS packets comply with the ASS spec.
> > 
> > Modified:
> >    trunk/libavformat/matroskadec.c
> > 
> 
> Imho, this requires a version bump as it changes the output format of matroska
> aas packets. (ie it will break some players which supported the old format). 

Previously, matroskadec didn't output ASS packets at all. It output something
similar to ASS but which was not ASS. This was a bug and this is now fixed.
No player should have ever relied on this bug to decode non-ASS packets...
Instead, people who implemented players should have reported the non-ASS
compliant packets bug here...
Still, I've just bumped micro version, in case it may help some players...

Aurel