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

Tomas Härdin 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 mailing list