[FFmpeg-devel] [PATCH]Support more AVC-Intra files

[Michael Niedermayer]

On Wed, Sep 26, 2012 at 07:21:18PM +0200, Reimar Döffinger wrote:
> On Wed, Sep 26, 2012 at 02:28:35PM +0200, Michael Niedermayer wrote:
> > On Wed, Sep 26, 2012 at 12:13:53PM +0200, Carl Eugen Hoyos wrote:
> > > Hi!
> > > 
> > > Followup to http://thread.gmane.org/gmane.comp.video.ffmpeg.devel/151321
> > > Attached patch by Reimar and Thomas Mundt fixes some AVC-Intra files from 
> > > different tickets.
> > 
> > > It does not fix http://samples.ffmpeg.org/ffmpeg-
> > > bugs/trac/ticket524/AVCI50.mov
> 
> That is probably due to progressive vs. interlaced detection issue,
> probably caused by the part that makes mov.c set field_order being
> removed.
> 
> > which file does it fix ?
> 
> I think it should fix all from tickets #1294, #524, #1666,
> at least if the above is fixed.

thanks


> 
> > > diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c
> > > index 16b8c12..1aff75b 100644
> > > --- a/libavformat/mxfdec.c
> > > +++ b/libavformat/mxfdec.c
> > 
> > mixing mxfdec and movdec changes in the same patch is bad
> 
> I disagree, they implement basically exactly the same feature in both,
> so it makes sense to do them together.

whichever way you prefer, i just suggested it because i was uncertain
about the mxf field_order change

thanks

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

He who knows, does not speak. He who speaks, does not know. -- Lao Tsu
-------------- 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/20120926/339c1d7b/attachment.asc>