[Ffmpeg-devel] Re: [PATCH] new mov_read_packet, mov_read_seek

[Baptiste Coudurier]

Hi

Michael Niedermayer wrote:
> [...]
> 
> and use it, there are so many places in libav* which need this that its
> starting to hurt to see this in various varations over and over ...
> 

Done

>> +    /* compute duration */
>> +    assert(sc->stts_data[sc->sample_to_time_index].duration % sc->time_rate == 0);
>> +    pkt->duration = sc->stts_data[sc->sample_to_time_index].duration / sc->time_rate;
> 
> i think this is wrong, if this is the dts difference then its not the real
> duration, AVPacket.duration must be the real duration, which is the pts
> difference
> 
> for example:
>          IPBBP
> dts:     01356
> pts:     16359
> duration:2321
> 

Applied a version without pkt->duration handling, finally. Now I can
step further.

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
SMARTJOG S.A.                                    http://www.smartjog.com
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
Phone: +33 1 49966312