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

Michael Niedermayer michaelni at gmx.at
Tue Dec 25 17:57:22 CET 2012


On Tue, Dec 25, 2012 at 05:12:04PM +0100, Tomas Härdin wrote:
> On Thu, 2012-12-20 at 16:01 +0100, Tomas Härdin wrote:
> > 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
> 
> ping?

patch applied

thanks

[...]

PS: if you have some "stable" branch in your git repo that contains
commits ready to be applied, this would prevent having patches missed.
I could simply merge that stable branch and all changes that you
felt where ready are then in ...

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Those who are best at talking, realize last or never when they are wrong.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20121225/6482a810/attachment.asc>


More information about the ffmpeg-devel mailing list