[FFmpeg-devel] [PATCH] MOV YUV2 fourcc -> PIX_FMT_YUYV422 mapping

Jai Menon jmenon86
Thu Apr 2 13:48:53 CEST 2009


On 4/2/09, Michael Niedermayer <michaelni at gmx.at> wrote:
> On Thu, Apr 02, 2009 at 10:46:31AM +0530, Jai Menon wrote:
>  > On 4/1/09, Michael Niedermayer <michaelni at gmx.at> wrote:
>  > > On Wed, Apr 01, 2009 at 02:34:10PM +0530, Jai Menon wrote:
>  > >  > On 4/1/09, Michael Niedermayer <michaelni at gmx.at> wrote:
>  > >  > > On Wed, Apr 01, 2009 at 10:23:52AM +0530, Jai Menon wrote:
>  > >  > >  > On 3/31/09, Michael Niedermayer <michaelni at gmx.at> wrote:
>  > >  > >  > > On Tue, Mar 31, 2009 at 10:10:28AM +0530, Jai Menon wrote:
>  > >  > >  > >  > On Mon, Mar 30, 2009 at 11:01 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
>  > >  > >  > >
>  > >  > >  > > [...]
>  > >  > >  > >  > revised patch attached.
>  > >  > >  > >  > --
>  > >  > >  > >  > Regards,
>  > >  > >  > >  >
>  > >  > >  > >  > Jai
>  > >  >
>  > >  > [...]
>  > >  >
>  > >  > > >  static int raw_encode(AVCodecContext *avctx,
>  > >  > >  >                              unsigned char *frame, int buf_size, void *data)
>  > >  > >  >  {
>  > >  > >  > -    return avpicture_layout((AVPicture *)data, avctx->pix_fmt, avctx->width,
>  > >  > >  > +    int ret = avpicture_layout((AVPicture *)data, avctx->pix_fmt, avctx->width,
>  > >  > >  >                                                 avctx->height, frame, buf_size);
>  > >  > >  > +
>  > >  > >
>  > >  > > > +    if(avctx->codec_tag == AV_RL32("yuv2") && ret > 0 &&
>  > >  > >  > +       avctx->pix_fmt   == PIX_FMT_YUYV422) {
>  > >  > >
>  > >  > > > +        int x, y;
>  > >  > >
>  > >  > >  > +        uint8_t *line = frame;
>  > >  > >
>  > >  > >
>  > >  > > redundant
>  > >  >
>  > >  > is attached patch okay?
>  > >
>  > >
>  > > if its tested and works, yes
>  > >
>  >
>  > 100l, i didn't correct the stride in rawenc.c hunk , correct and
>  > tested patch attached.
>
>
> ok

[...]

Thanks for the review. Could someone please apply?
Also, I guess qualification task is done then.

-- 
Regards,

Jai



More information about the ffmpeg-devel mailing list