[FFmpeg-devel] [PATCH] lavf/concatdec: fix timestamp conversion

[Nicolas George]

Le nonidi 29 brumaire, an CCXXIII, Stefano Sabatini a écrit :
> I'm concatenating samples with different time bases. Apparently
> concatdec is setting the first stream timebase as the overall
> timebase, and always assuming the timebase is shared for all the input
> files.

Concatenating segments with different settings is not currently supported,
and that includes the time base. But inconsistencies are not detected
either.

For the time base specifically, I suppose implementing the conversion and a
smart time base selection would be enough. With just the conversion, you
might get lucky if the time base of the first segment is accurate enough.

I do not object to the patch implementing the conversion, it will be needed
at some point anyway to support segments with different settings, but a
comment that this is incomplete would probably be a useful remainder.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20141119/c7ee8120/attachment.asc>