[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