[FFmpeg-devel] [PATCH] Fix decoding of DNxHD video in MXF container
michaelni at gmx.at
Sun Mar 11 17:46:54 CET 2012
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
Size: 198 bytes
Desc: Digital signature
More information about the ffmpeg-devel