[FFmpeg-devel] Official status of FFv1?

[Michael Niedermayer]

On Mon, Mar 22, 2010 at 07:24:19PM +0100, Peter B. wrote:
> About FFv1 and colorspaces:
> 
> I'm not quite sure how to interpret the sourcecode.
> It looks like it's able to cope with several colorspaces - and although
> the case-statement (in ffv1.c / line #667) looks like it groups some
> colorspaces together, I propose it simply uses it to distinguish between
> RGB and YUV color models (and 8/16 bit), later on at line #755.
> 
> Here's the switch-block I'm talking about:
> ---------------------------------------------------------
> switch(avctx->pix_fmt){
>     case PIX_FMT_YUV444P16:
>     case PIX_FMT_YUV422P16:
>     case PIX_FMT_YUV420P16:
>         if(avctx->strict_std_compliance > FF_COMPLIANCE_EXPERIMENTAL){
>             av_log(avctx, AV_LOG_ERROR, "More than 8 bit per component
> is still experimental and no gurantee is yet made for future
> compatibility\n"
>                "Use vstrict=-2 / -strict -2 to use it anyway.\n");
>             return -1;
>         }
>         if(avctx->bits_per_raw_sample <=8){
>             av_log(avctx, AV_LOG_ERROR, "bits_per_raw_sample invalid\n");
>             return -1;
>         }
>         s->version= 1;
>     case PIX_FMT_YUV444P:
>     case PIX_FMT_YUV422P:
>     case PIX_FMT_YUV420P:
>     case PIX_FMT_YUV411P:
>     case PIX_FMT_YUV410P:
>         s->colorspace= 0;
>         break;
>     case PIX_FMT_RGB32:
>         s->colorspace= 1;
>         break;
>     default:
>         av_log(avctx, AV_LOG_ERROR, "format not supported\n");
>         return -1;
>     }
> ---------------------------------------------------------
> 
> So, if I understood correctly, FFV1 *is* able to properly handle all
> colorspaces mentioned in the statement above, right?

yes, the following are supported

PIX_FMT_YUV420P
PIX_FMT_YUV444P
PIX_FMT_YUV422P
PIX_FMT_YUV411P
PIX_FMT_YUV410P
PIX_FMT_RGB32
PIX_FMT_YUV420P16
PIX_FMT_YUV422P16
PIX_FMT_YUV444P16

adding more is a fairly trivial

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

Good people do not need laws to tell them to act responsibly, while bad
people will find a way around the laws. -- Plato
-------------- 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/20100322/2f92ae16/attachment.pgp>