[FFmpeg-cvslog] r18959 - in trunk: libavcodec/mpeg12.c tests/seek.regression.ref
Baptiste Coudurier
baptiste.coudurier
Wed May 27 08:38:19 CEST 2009
On 5/26/2009 9:40 PM, 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
Yes, that's because now frames have pts correctly set :)
This PVA samples does not start with a SEQ EXT therefore first frames
dts are not computed accurately.
Btw, any reason for not setting has_b_frames when a B frame is
encountered or I frame with temp_ref != 0 ?
Patch attached.
Btw should framecrc muxer have AVFMT_NOTIMESTAMPS set, though I wouldn't
catch the failure in libavormat itself but the crc would differ, not
sure about that.
--
Baptiste COUDURIER GnuPG Key Id: 0x5C1ABAAA
Key fingerprint 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
FFmpeg maintainer http://www.ffmpeg.org
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: parser_b_frames.patch
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20090526/d6b80283/attachment.asc>
More information about the ffmpeg-cvslog
mailing list