[FFmpeg-devel] [PATCH] Fill unspecified AVCodec long_name fields

[Diego Biurrun]

On Sat, Apr 26, 2008 at 01:00:56PM +0200, Stefano Sabatini wrote:
> 
> Thanks, new patch attached.
> 
> --- libavcodec/8bps.c	(revision 12974)
> +++ libavcodec/8bps.c	(working copy)
> @@ -230,4 +230,5 @@
>          CODEC_CAP_DR1,
> +        .long_name = "Quicktime 8BPS Video",

QuickTime 8BPS video

> --- libavcodec/apedec.c	(revision 12974)
> +++ libavcodec/apedec.c	(working copy)
> @@ -917,4 +917,5 @@
>      ape_decode_frame,
> +    .long_name = "Monkey's Audio lossless audio",

Monkey's Audio

> --- libavcodec/dsicinav.c	(revision 12974)
> +++ libavcodec/dsicinav.c	(working copy)
> @@ -348,6 +348,7 @@
>      CODEC_CAP_DR1,
> +    .long_name = "Delphine Software International CIN Video",

video

> @@ -359,4 +360,5 @@
>      cinaudio_decode_frame,
> +    .long_name = "Delphine Software International CIN Audio",

audio

> --- libavcodec/dvbsub.c	(revision 12974)
> +++ libavcodec/dvbsub.c	(working copy)
> @@ -408,4 +408,5 @@
>      dvbsub_encode,
> +    .long_name = "DVB Subtitles",

subtitles

> --- libavcodec/dvbsubdec.c	(revision 12974)
> +++ libavcodec/dvbsubdec.c	(working copy)
> @@ -1429,4 +1429,5 @@
>      dvbsub_decode,
> +    .long_name = "DVB Subtitles",

subtitles

> --- libavcodec/dvdsubdec.c	(revision 12974)
> +++ libavcodec/dvdsubdec.c	(working copy)
> @@ -506,4 +506,5 @@
>      dvdsub_decode,
> +    .long_name = "DVD Subtitles",

subtitles

> --- libavcodec/dvdsubenc.c	(revision 12974)
> +++ libavcodec/dvdsubenc.c	(working copy)
> @@ -222,4 +222,5 @@
>      dvdsub_encode,
> +    .long_name = "DVD Subtitles",

subtitles

> --- libavcodec/dxa.c	(revision 12974)
> +++ libavcodec/dxa.c	(working copy)
> @@ -326,6 +326,7 @@
>      decode_end,
> +    .long_name = "Feeble Files/ScummVM DXA decoder",

s/decoder//

> --- libavcodec/ffv1.c	(revision 12974)
> +++ libavcodec/ffv1.c	(working copy)
> @@ -1035,5 +1035,6 @@
>      .pix_fmts= (enum PixelFormat[]){PIX_FMT_YUV420P, PIX_FMT_YUV444P, PIX_FMT_YUV422P, PIX_FMT_YUV411P, PIX_FMT_YUV410P, PIX_FMT_RGB32, -1},
> +    .long_name= "FF Video Codec 1 (an experimental lossless codec)"

Hmmm, the multimedia wiki claims that the name is "FFmpeg codec #1".

> --- libavcodec/flashsv.c	(revision 12974)
> +++ libavcodec/flashsv.c	(working copy)
> @@ -255,4 +255,5 @@
>      .pix_fmts = (enum PixelFormat[]){PIX_FMT_BGR24, -1},
> +    .long_name = "Flash Screen Video",

video

> --- libavcodec/flashsvenc.c	(revision 12974)
> +++ libavcodec/flashsvenc.c	(working copy)
> @@ -293,5 +293,6 @@
>      .pix_fmts = (enum PixelFormat[]){PIX_FMT_BGR24, -1},
> +    .long_name = "Flash Screen Video",

video

> --- libavcodec/flicvideo.c	(revision 12974)
> +++ libavcodec/flicvideo.c	(working copy)
> @@ -749,5 +749,6 @@
> +    .long_name = "Autodesk Animator Flic (.fli/.flc) Video"

video

> --- libavcodec/huffyuv.c	(revision 12974)
> +++ libavcodec/huffyuv.c	(working copy)
> @@ -1457,7 +1458,8 @@
> +    NULL,
> +    .long_name = "FFmpeg's Variant of Huffyuv",

variant

> @@ -1483,6 +1486,7 @@
>      .pix_fmts= (enum PixelFormat[]){PIX_FMT_YUV420P, PIX_FMT_YUV422P, PIX_FMT_RGB32, -1},
> +    .long_name = "FFmpeg's Variant of Huffyuv",

variant

> --- libavcodec/kmvc.c	(revision 12974)
> +++ libavcodec/kmvc.c	(working copy)
> @@ -410,5 +410,6 @@
> +    decode_frame,
> +    .long_name = "Karl Morton's Video Codec",

video codec

> --- libavcodec/lcldec.c	(revision 12974)
> +++ libavcodec/lcldec.c	(working copy)
> @@ -711,5 +712,6 @@
>      CODEC_CAP_DR1,
> +    .long_name ="LCL (LossLess Codec Library) ZLIB",
> --- libavcodec/lclenc.c	(revision 12974)
> +++ libavcodec/lclenc.c	(working copy)
> @@ -228,4 +228,5 @@
>      encode_end,
> +    .long_name = "LCL (LossLess Codec Library) Video Codec ZLIB",

The first variant is better.

> --- libavcodec/mmvideo.c	(revision 12974)
> +++ libavcodec/mmvideo.c	(working copy)
> @@ -202,4 +202,5 @@
>      CODEC_CAP_DR1,
> +    .long_name = "American Laser Games MM Video",

video

> --- libavcodec/qdm2.c	(revision 12974)
> +++ libavcodec/qdm2.c	(working copy)
> @@ -2038,4 +2038,5 @@
>      .decode = qdm2_decode_frame,
> +    .long_name = "QDM2 2 (QDesign Music Codec 2)",

Leave out the fourcc and the extra "2".

> --- libavcodec/qtrle.c	(revision 12974)
> +++ libavcodec/qtrle.c	(working copy)
> @@ -620,5 +620,6 @@
>      CODEC_CAP_DR1,
> +    .long_name = "Quicktime Animation (RLE) Video",

QuickTime, video

> --- libavcodec/qtrleenc.c	(revision 12974)
> +++ libavcodec/qtrleenc.c	(working copy)
> @@ -323,4 +323,5 @@
>      .pix_fmts = (enum PixelFormat[]){PIX_FMT_RGB24, -1},
> +    .long_name = "Quicktime Animation (RLE) Video",

ditto

> --- libavcodec/rawdec.c	(revision 12974)
> +++ libavcodec/rawdec.c	(working copy)
> @@ -162,4 +162,5 @@
>      raw_decode,
> +    .long_name = "Raw video",
> --- libavcodec/rawenc.c	(revision 12974)
> +++ libavcodec/rawenc.c	(working copy)
> @@ -51,4 +51,5 @@
>      raw_encode,
> +    .long_name = "Raw video",

raw

> --- libavcodec/roqaudioenc.c	(revision 12974)
> +++ libavcodec/roqaudioenc.c	(working copy)
> @@ -174,4 +174,5 @@
>      NULL,
> +    .long_name = "Id RoQ Audio",

audio

> --- libavcodec/roqvideodec.c	(revision 12974)
> +++ libavcodec/roqvideodec.c	(working copy)
> @@ -219,4 +219,5 @@
>      CODEC_CAP_DR1,
> +    .long_name = "Id RoQ Video",

video

> --- libavcodec/roqvideoenc.c	(revision 12974)
> +++ libavcodec/roqvideoenc.c	(working copy)
> @@ -1066,4 +1066,5 @@
>      .pix_fmts = (enum PixelFormat[]){PIX_FMT_YUV444P, -1},
> +    .long_name = "Id RoQ Video",

video

> --- libavcodec/rpza.c	(revision 12974)
> +++ libavcodec/rpza.c	(working copy)
> @@ -283,4 +283,5 @@
>      CODEC_CAP_DR1,
> +    .long_name = "Quicktime Video (RPZA)",

QuickTime video (RPZA)

Are you sure this name is correct?

> --- libavcodec/rv10.c	(revision 12974)
> +++ libavcodec/rv10.c	(working copy)
> @@ -784,7 +784,8 @@
> +    CODEC_CAP_DR1,
> +    .long_name = "RV10 (Real Video version 10)",

Leave out the fourcc, 1.0.

> @@ -798,5 +799,6 @@
>      .flush= ff_mpeg_flush,
> +    .long_name = "RV20 (Real Video version 20)",

ditto

> --- libavcodec/rv30.c	(revision 12974)
> +++ libavcodec/rv30.c	(working copy)
> @@ -144,4 +144,5 @@
>      CODEC_CAP_DR1 | CODEC_CAP_DELAY,
> +    .long_name = "RV30 (Real Video version 30)",
> --- libavcodec/rv40.c	(revision 12974)
> +++ libavcodec/rv40.c	(working copy)
> @@ -276,4 +276,5 @@
>      CODEC_CAP_DR1 | CODEC_CAP_DELAY,
> +    .long_name = "RV40 (Real Video version 40)",

ditto

> --- libavcodec/smacker.c	(revision 12974)
> +++ libavcodec/smacker.c	(working copy)
> @@ -697,7 +697,8 @@
> +    decode_frame,
> +    .long_name = "Smacker Video",

video

> @@ -708,6 +709,7 @@
> +    smka_decode_frame,
> +    .long_name = "Smacker Audio",

audio

> --- libavcodec/smc.c	(revision 12974)
> +++ libavcodec/smc.c	(working copy)
> @@ -484,4 +484,5 @@
>      CODEC_CAP_DR1,
> +    .long_name = "Quicktime Graphics (SMC)",

QuickTime

> --- libavcodec/sunrast.c	(revision 12974)
> +++ libavcodec/sunrast.c	(working copy)
> @@ -191,5 +191,6 @@
> +    NULL,
> +    .long_name = "Sun Rasterfile (.sun/.ras/im{1,8,24}/.sunras) image",

Leave out the filename extensions.

> --- libavcodec/svq1dec.c	(revision 12974)
> +++ libavcodec/svq1dec.c	(working copy)
> @@ -826,4 +826,5 @@
>      .pix_fmts= (enum PixelFormat[]){PIX_FMT_YUV410P, -1},
> +    .long_name= "SVQ1 (Sorensen Vector Quantizer 1)",
> --- libavcodec/svq1enc.c	(revision 12974)
> +++ libavcodec/svq1enc.c	(working copy)
> @@ -585,4 +585,5 @@
>      .pix_fmts= (enum PixelFormat[]){PIX_FMT_YUV410P, -1},
> +    .long_name= "SVQ1 (Sorensen Vector Quantizer 1)",
> --- libavcodec/svq3.c	(revision 12974)
> +++ libavcodec/svq3.c	(working copy)
> @@ -982,4 +982,5 @@
>      CODEC_CAP_DRAW_HORIZ_BAND | CODEC_CAP_DR1 | CODEC_CAP_DELAY,
> +    .long_name = "SVQ3 (Sorensen Vector Quantizer 3)",

SorensOn, leave out the fourccs.

> --- libavcodec/zmbv.c	(revision 12974)
> +++ libavcodec/zmbv.c	(working copy)
> @@ -661,6 +661,7 @@
> +    decode_frame,
> +    .long_name = "Zip Motion Blocks Video (ZMBV)",

Leave out the fourcc.

> --- libavcodec/zmbvenc.c	(revision 12974)
> +++ libavcodec/zmbvenc.c	(working copy)
> @@ -319,4 +319,5 @@
>      .pix_fmts = (enum PixelFormat[]){PIX_FMT_PAL8, -1},
> +    .long_name = "Zip Motion Blocks Video (ZMBV)",

ditto

Diego