[FFmpeg-cvslog] r18959 - in trunk: libavcodec/mpeg12.c tests/seek.regression.ref

[Reimar Döffinger]

On Tue, May 26, 2009 at 09:40:46PM -0700, Mike Melanson wrote:
> bcoudurier wrote:
> > Author: bcoudurier
> > Date: Wed May 27 02:14:32 2009
> > New Revision: 18959
> > 
> > Log:
> > Fix off by one offset with fetch_timestamps, pts_parser_problem.mpg.
> > Patch by Wolfram Gloger, wmglo at dentm dot med dot uni-muenchen dot de.
> > 
> > Modified:
> >    trunk/libavcodec/mpeg12.c
> >    trunk/tests/seek.regression.ref
> 
> Do you have any idea why this might have broken the pva-demux test?
> 
> http://fate.multimedia.cx/index.php?test_spec=298
> 
> The file errors out with the following message:
> 
> [NULL @ 0x120ab5730]error, non monotone timestamps 50151 >= 42951
> av_interleaved_write_frame(): Error while opening file

I don't like it much, but since FATE staying broken or disabling the
test seems even worse:
How about changing the test to be based on the
http://samples.mplayerhq.hu/PVA/_B5_Crusade_-_Kosmiczna_Krucjata_01pl_1276.pva
or
http://samples.mplayerhq.hu/PVA/_Babylon_5_5x01pl_1251.pva
sample?
That leaves 5/7 of our sample files broken but better than nothing...