[FFmpeg-devel] [PATCH] mxfdec: Rescale audio stream duration from EditRate to SampleRate

Tomas Härdin tomas.hardin at codemill.se
Thu Dec 20 16:01:35 CET 2012


On Fri, 2012-12-14 at 15:42 +0100, Tomas Härdin wrote:
> Fixes https://ffmpeg.org/trac/ffmpeg/ticket/2029

Apparently that patch breaks FATE. New patch attached which also fixes
two previously unnoticed seek issues in mxfdec, namely:

>  ret: 0         st: 1 flags:1  ts: 0.200833
> -ret: 0         st: 0 flags:1 dts: 0.840000 pts: 0.960000 pos: 460288 size: 24711
> +ret: 0         st: 0 flags:1 dts:-0.040000 pts: 0.000000 pos:   6144 size: 24801

Since flags=1 (AVSEEK_FLAG_BACKWARD) previously we'd end up ahead of ts.
After this patch the demuxer seeks before ts as expected.

Also:

>  ret: 0         st: 1 flags:1  ts: 0.200833
> -ret: 0         st: 0 flags:1 dts: 0.000000 pts: 0.000000 pos:5117952 size:150000
> +ret: 0         st: 0 flags:1 dts: 0.200000 pts: 0.200000 pos:1071104 size:150000

seems like a more reasonable place to end up at.

/Tomas
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-mxfdec-Rescale-audio-stream-duration-from-EditRate-t.patch
Type: text/x-patch
Size: 5240 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20121220/4c39f171/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20121220/4c39f171/attachment.asc>


More information about the ffmpeg-devel mailing list