[FFmpeg-devel] [PATCH] Fix decoding of DNxHD video in MXF container

[Michael Niedermayer]

On Wed, Feb 08, 2012 at 09:41:04AM +0100, Tomas Härdin wrote:
> On Tue, 2012-02-07 at 14:55 +0000, Joseph Artsimovich wrote:
> > On 07/02/2012 14:47, Joseph Artsimovich wrote:
> > > Updated versions of my patches are attached.
> > > The old DNxHD signature is retained and MXFFrameLayout enum was 
> > > introduced.
> > Sent the wrong version that wouldn't build.  Correct version attached.
> 
> > 
> > +                    break; /* The correct thing to do here is fall through, but by breaking we might be
> > +                    able to decode some streams at half the vertical resolution, rather than not al all.
> > +                    It's also for compatibility with the old behavior. */
> 
> Strange indentation makes this a little hard to read.
> Both LGTM otherwise.

First patch applied

second patch seems to have been applied already

Thanks to Joseph for the patch and baptiste and tomas for the review
and sorry for the delay, i was hoping someone else would be fixing
the indention so i didnt had to do it ...

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

The real ebay dictionary, page 3
"Rare item" - "Common item with rare defect or maybe just a lie"
"Professional" - "'Toy' made in china, not functional except as doorstop"
"Experts will know" - "The seller hopes you are not an expert"
-------------- 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/20120311/34dee823/attachment.asc>