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

[Michael Niedermayer]

On Tue, Mar 31, 2009 at 10:57:21AM -0700, Baptiste Coudurier wrote:
> On 3/31/2009 8:40 AM, Michael Niedermayer 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
> > 
> >> Index: libavcodec/rawenc.c
> >> ===================================================================
> >> --- libavcodec/rawenc.c	(revision 17944)
> >> +++ libavcodec/rawenc.c	(working copy)
> >> @@ -40,8 +40,20 @@
> >>  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 == MKTAG('y', 'u', 'v', '2') && (ret > 0) &&
> >> +       (avctx->pix_fmt == PIX_FMT_YUYV422)) {
> > 
> > superflous ()
> > AV_RL32("yuv2")
> 
> Interesting, should we use this now ?
> It would greatly improve readability IMHO.

we already use it at some places, namely h263.c h263dec.c

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Into a blind darkness they enter who follow after the Ignorance,
they as if into a greater darkness enter who devote themselves
to the Knowledge alone. -- Isha Upanishad
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090331/06330649/attachment.pgp>