[FFmpeg-devel] [PATCH 3/5] mxfdec: Clip wrapping and seeking support

[Michael Niedermayer]

On Mon, Nov 21, 2011 at 11:43:21AM +0100, Tomas Härdin wrote:
> On Fri, 2011-11-18 at 15:53 +0100, Michael Niedermayer wrote:
> > Hi
> > 
> > On Fri, Nov 11, 2011 at 03:45:58PM +0100, Tomas Härdin wrote:
> > > 
> > 
> > >  mxfdec.c |  113 +++++++++++++++++++++++++++++++++++++++++++++++++++------------
> > >  1 file changed, 92 insertions(+), 21 deletions(-)
> > > 1f325e0c5dfe2557025569c0d79ad565c32aeef8  0003-mxfdec-Clip-wrapping-and-seeking-support.patch
> > > From a6d67e1ab3af7d24a297dc2a42405226f935578f Mon Sep 17 00:00:00 2001
> > > From: Georg Lippitsch <georg.lippitsch at gmx.at>
> > > Date: Wed, 5 Oct 2011 11:05:16 +0200
> > > Subject: [PATCH 3/5] mxfdec: Clip wrapping and seeking support
> > 
> > This breaks make fate-mxf-demux:
> > 
> > [mxf @ 0x2186a20] unknown operational pattern: 40h 01h
> >     Last message repeated 1 times
> > /home/michael/fate/fate-suite//mxf/C0023S01.mxf: Operation not permitted
> 
> Fixed. Patch 5/5 needed conflict resolution after, so I'm going to post
> an updated version of it too.
> I also changed the "if (sample_time < 0) sample_time = 0;" thing to use
> FFMAX per patcheck's suggestion.
> 
> I'm aware this patch changes the results of the seek test, which I'm not
> quite sure how to adjust.

if the change is correct you can just change
tests/ref/seek/lavf_mxf / tests/ref/seek/lavf_mxf_d10
and it should end in the patch git generates


> We should also add two OPAtom files to FATE
> (one audio, one video).

yes, please contact baptiste about adding files to fate

Thanks

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The real ebay dictionary, page 1
"Used only once"    - "Some unspecified defect prevented a second use"
"In good condition" - "Can be repaird by experienced expert"
"As is" - "You wouldnt want it even if you were payed for it, if you knew ..."
-------------- 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/20111121/cc272397/attachment.asc>