[FFmpeg-cvslog] r22802 - trunk/libavformat/utils.c

Vitor Sessak vitor1001
Tue Apr 13 03:21:27 CEST 2010


Michael Niedermayer wrote:
> On Mon, Apr 12, 2010 at 08:12:50PM -0400, Vitor Sessak wrote:
>> Michael Niedermayer wrote:
>>> On Sun, Apr 04, 2010 at 09:40:02PM -0400, Vitor Sessak wrote:
>>>> michael wrote:
>>>>> Author: michael
>>>>> Date: Mon Apr  5 00:19:42 2010
>>>>> New Revision: 22802
>>>>> Log:
>>>>> Fix rounding direction for calculation of AVPacket.duration.
>>>>> Fixes issue1579
>>>> This broke "make fate-nsv-demux" and "make fate-vqf-demux".
>>> both the nsv and vqf demuxers set audio timebases in which audio frame
>>> timestamps cannot be represented accurately, in nsvs case the effect is
>>> a negligible +-1 change. For vqf the effect is a fatal growing loss of
>>> precission.
>>> For both files the bug is in the demuxers and exists before 22802.
>>> Also both pre 22802 and post 22802 checksums thus are equally wrong
>>> The demuxers should be fixed by their respective maintainers
>> I suppose like this?
>>
>> -Vitor
> 
>>  vqf.c |    1 +
>>  1 file changed, 1 insertion(+)
>> b340cee6c3185958d950a45741c35d58ebfe7f57  vqf_pts.diff
> 
> yes, looks correct

Applied. @Mike: Can you update the testspec?

Any idea of how to fix it in NSV demuxer where the sample rate is not 
encoded in the header?

-Vitor



More information about the ffmpeg-cvslog mailing list