[FFmpeg-cvslog] r16127 - trunk/libavformat/avidec.c

[Michael Niedermayer]

On Tue, Dec 16, 2008 at 02:22:36PM +0100, Michael Niedermayer wrote:
> On Tue, Dec 16, 2008 at 08:42:52AM +0200, Kostya wrote:
> > On Tue, Dec 16, 2008 at 03:48:32AM +0100, Michael Niedermayer wrote:
> > > On Sun, Dec 14, 2008 at 08:23:04PM -0800, Mike Melanson wrote:
> > > > Michael Niedermayer wrote:
> > > > > On Sun, Dec 14, 2008 at 10:33:49AM -0800, Mike Melanson wrote:
> > > > >> michael wrote:
> > > > >>> Author: michael
> > > > >>> Date: Sun Dec 14 16:48:48 2008
> > > > >>> New Revision: 16127
> > > > >>>
> > > > >>> Log:
> > > > >>> Fix non interleaved variable fps ODML avis.
> > > > >>> Fixed issue738
> > > > >> Problem: This causes an infinite loop when there is only 1 frame in an 
> > > > >> AVI file. This causes FATE to stall on this test:
> > > > >>
> > > > >> http://fate.multimedia.cx/index.php?test_spec=124
> > > > >>
> > > > >> (Problem file is linked from there.) The stall occurs on Linux 
> > > > >> x86_{32|64} but not on Linux/PPC or either Mac OS X configuration.
> > > > > 
> > > > > should be fixed
> > > > 
> > > > BTW, this change also caused the following test spec to fail on all 
> > > > configurations (I bisected it to revision 16127):
> > > > 
> > > > http://fate.multimedia.cx/index.php?test_spec=252
> > > > 
> > > > $BUILD_PATH/ffmpeg -i $SAMPLES_PATH/VMnc/VS2k5DebugDemo-01-partial.avi 
> > > > -vsync 0 -pix_fmt rgb24 -f framecrc -
> > > > 
> > > > Sample is linked from page. Is there a reason that the CRC output from 
> > > > the decoder would have changed? Note that this is the case with or 
> > > > without '-vsync 0'.
> > > 
> > > the demuxer output changed like:
> > > --- vold	2008-12-16 03:46:17.000000000 +0100
> > > +++ old	2008-12-16 03:47:42.000000000 +0100
> > > @@ -1,4 +1,3 @@
> > > -0, 0, 1655, 0x989d4766
> > >  0, 18000, 1655, 0x989d4766
> > >  0, 36000, 22, 0x2375044f
> > >  0, 90000, 1673, 0x2024483e
> > > 
> > > Thus it seems to me that te demuxer previously returned the first packet
> > > twice.
> > > Comment by the decoder maintainer is welcome, but without further
> > > investigation i suspect it was wrong and is more correct now.
> > 
> > MPlayer plays it fine while ffmpeg cannot find first keyframe and decodes
> > the differences only.
> 
> hmm, ill look into this later today. But there was no keyframe in the
> previous demuxer output either unless i messed up so it is wrong but
> the previous was wrong as well.

should be fixed

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Freedom in capitalist society always remains about the same as it was in
ancient Greek republics: Freedom for slave owners. -- Vladimir Lenin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20081216/a2e9a0b0/attachment.pgp>