[FFmpeg-devel] [PATCH 2/2] mxfenc: support dnxhd codec

Tomas Härdin tomas.hardin at codemill.se
Mon Jul 2 16:30:48 CEST 2012


On Sat, 2012-06-30 at 00:44 +0200, Matthieu Bouron wrote:
> > +    switch (cid) {
> > +    case 1235:
> > +        sc->index = 24;
> > +        sc->component_depth = 10;
> > +        break;
> > +    case 1237:
> > +        sc->index = 25;
> > +        break;
> > +    case 1238:
> > +        sc->index = 26;
> > +        break;
> > +    case 1241:
> > +        sc->index = 27;
> > +        sc->component_depth = 10;
> > +        break;
> > +    case 1242:
> > +        sc->index = 28;
> > +        break;
> > +    case 1243:
> > +        sc->index = 29;
> > +        break;
> > +    case 1250:
> > +        sc->index = 30;
> > +        sc->component_depth = 10;
> > +        break;
> > +    case 1251:
> > +        sc->index = 31;
> > +        break;
> > +    case 1252:
> > +        sc->index = 32;
> > +        break;
> > +    case 1253:
> > +        sc->index = 33;
> > +        break;
> > +    default:
> > +        return -1;
> > +    }

I did a quick double-check against RP224 for these (cid vs UL indexed)
and they seem correct (more of a note).

The patch looks OK to me.

/Tomas



More information about the ffmpeg-devel mailing list