[FFmpeg-cvslog] r23074 - trunk/libavcodec/raw.c

[Stefano Sabatini]

On date Friday 2010-06-04 16:00:56 -0700, Baptiste Coudurier wrote:
> On 05/09/2010 04:04 PM, stefano wrote:
[...]
> >Modified: trunk/libavcodec/raw.c
> >==============================================================================
> >--- trunk/libavcodec/raw.c	Mon May 10 01:04:01 2010	(r23073)
> >+++ trunk/libavcodec/raw.c	Mon May 10 01:04:03 2010	(r23074)
> >@@ -69,6 +69,16 @@ const PixelFormatTag ff_raw_pixelFormatT
> >      { PIX_FMT_RGB565LE, MKTAG('R', 'G', 'B', 16) },
> >      { PIX_FMT_BGR565LE, MKTAG('B', 'G', 'R', 16) },
> >      { PIX_FMT_RGB565LE, MKTAG( 3 ,  0 ,  0 ,  0) },
> >+    { PIX_FMT_RGB555BE, MKTAG(15 , 'B', 'G', 'R') },
> >+    { PIX_FMT_BGR555BE, MKTAG(15 , 'R', 'G', 'B') },
> >+    { PIX_FMT_RGB565BE, MKTAG(16 , 'B', 'G', 'R') },
> >+    { PIX_FMT_BGR565BE, MKTAG(16 , 'R', 'G', 'B') },
> >+    { PIX_FMT_RGBA,     MKTAG('R', 'G', 'B', 'A') },
> >+    { PIX_FMT_BGRA,     MKTAG('B', 'G', 'R', 'A') },
> >+    { PIX_FMT_ABGR,     MKTAG('A', 'B', 'G', 'R') },
> >+    { PIX_FMT_ARGB,     MKTAG('A', 'R', 'G', 'B') },
> >+    { PIX_FMT_RGB24,    MKTAG('R', 'G', 'B', 24 ) },
> >+    { PIX_FMT_BGR24,    MKTAG('B', 'G', 'R', 24 ) },
> 
> Btw, quicktime has fourcc for these:
> L565
> L555
> B565
> 
> see movenc.c
> 
> we may want to reuse these instead. What do you guys think ?

Then they should be placed before the nut tags, otherwise this is
going to break MOV encoding.

BTW we should keep somewhere a complete tag list, fourcc.org looks
undermaintained and doesn't include MOV tags anyway.

Regards.