[FFmpeg-devel] [PATCH] Fix decoding of DNxHD video in MXF container
tomas.hardin at codemill.se
Wed Feb 8 09:41:04 CET 2012
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.
More information about the ffmpeg-devel