[FFmpeg-devel] [PATCH] matroskadec: Fix TTA duration calculation

Paul B Mahol onemda at gmail.com
Thu Aug 15 12:52:00 CEST 2013


On 8/15/13, Paul B Mahol <onemda at gmail.com> wrote:
> On 8/15/13, Michael Niedermayer <michaelni at gmx.at> wrote:
>> On Thu, Aug 15, 2013 at 06:22:47AM -0300, James Almer wrote:
>>> Still not perfect due to rounding errors, but several samples should
>>> demux correctly now instead of none.
>>
>> please add a link to a sample to reproduce this bugfix in the commit
>> message
>
> Samples are not needed at all, this can be easily reproduced.
>
> And its incorrect to claim current one could not demux correctly any files.
> It just fails if last packet have number of samples smaller than all other
> packets before, which is 'almost' always but not always.
>
> Question to OP: I wonder does other program(s) always decode last frame no
> matter what number of samples are in it?
>

Also commit message is misleading: it does not fix duration
calculation it just improves it.
AFAIK there is no way to really fix calculation, but I would like to be proven
wrong....


More information about the ffmpeg-devel mailing list