[FFmpeg-devel] [PATCH] Fill unspecified AVCodec long_name fields
Kostya
kostya.shishkov
Sat Apr 26 06:31:17 CEST 2008
On Sat, Apr 26, 2008 at 02:45:17AM +0200, Stefano Sabatini wrote:
[...]
> Mmmh... I prefer the prefix for consistency furthermore the
> abovementioned codecs are library interfaces, so IMHO they should
> explicitly refer to the library used (no strong opinion on this, so if
> you or other insist then I'll change it).
>
> Addressed all the other comments, also I added some missing long_names
> and fixed some wrong ones.
>
> Best regards.
> --
> FFmpeg = Friendly & Frenzy MultiPurpose EnGine
Now it's my turn to criticize ;)
> --- libavcodec/aasc.c (revision 12974)
> +++ libavcodec/aasc.c (working copy)
> @@ -171,4 +171,5 @@
> aasc_decode_end,
> aasc_decode_frame,
> CODEC_CAP_DR1,
> + .long_name = "Autodesk RLE",
> };
Ehm, spotted after ~2.5 years :)
ok
[...]
> Index: libavcodec/apedec.c
> ===================================================================
> --- libavcodec/apedec.c (revision 12974)
> +++ libavcodec/apedec.c (working copy)
> @@ -917,4 +917,5 @@
> NULL,
> ape_decode_close,
> ape_decode_frame,
> + .long_name = "Monkey's Audio lossless audio",
> };
ok
[...]
> Index: libavcodec/cljr.c
> ===================================================================
> --- libavcodec/cljr.c (revision 12974)
> +++ libavcodec/cljr.c (working copy)
> @@ -140,6 +140,7 @@
> NULL,
> decode_frame,
> CODEC_CAP_DR1,
> + .long_name = "Cirrus Logic AccuPak (CLJR)",
> };
> #if 0
> #ifdef CONFIG_ENCODERS
> @@ -152,6 +153,7 @@
> encode_init,
> encode_frame,
> //encode_end,
> + .long_name = "Cirrus Logic AccuPak (CLJR)",
> };
>
> #endif //CONFIG_ENCODERS
Not mine, but fourcc in () seems a bit unneeded
[...]
> Index: libavcodec/dxa.c
> ===================================================================
> --- libavcodec/dxa.c (revision 12974)
> +++ libavcodec/dxa.c (working copy)
> @@ -326,6 +326,7 @@
> decode_init,
> NULL,
> decode_end,
> - decode_frame
> + decode_frame,
> + .long_name = "Feeble Files/ScummVM DXA decoder",
> };
ok
[...]
> Index: libavcodec/flicvideo.c
> ===================================================================
> --- libavcodec/flicvideo.c (revision 12974)
> +++ libavcodec/flicvideo.c (working copy)
> @@ -749,5 +749,6 @@
> NULL,
> NULL,
> NULL,
> - NULL
> + NULL,
> + .long_name = "FLI/FLC Animation",
> };
IIRC, it's called "Flic"
[...]
> Index: libavcodec/fraps.c
> ===================================================================
> --- libavcodec/fraps.c (revision 12974)
> +++ libavcodec/fraps.c (working copy)
> @@ -365,4 +365,5 @@
> decode_end,
> decode_frame,
> CODEC_CAP_DR1,
> + .long_name = "Fraps FPS1",
> };
Maybe simply "Fraps"?
[...]
> Index: libavcodec/indeo3.c
> ===================================================================
> --- libavcodec/indeo3.c (revision 12974)
> +++ libavcodec/indeo3.c (working copy)
> @@ -1134,5 +1134,6 @@
> indeo3_decode_end,
> indeo3_decode_frame,
> 0,
> - NULL
> + NULL,
> + .long_name = "Intel Indeo 3 (IV31, IV32, etc.)",
> };
Fourccs again
[...]
> Index: libavcodec/jpeglsenc.c
> ===================================================================
> --- libavcodec/jpeglsenc.c (revision 12974)
> +++ libavcodec/jpeglsenc.c (working copy)
> @@ -390,4 +390,5 @@
> encode_picture_ls,
> NULL,
> .pix_fmts= (enum PixelFormat[]){PIX_FMT_BGR24, PIX_FMT_RGB24, PIX_FMT_GRAY8, PIX_FMT_GRAY16, -1},
> + .long_name= "JPEG-LS",
> };
ok
> Index: libavcodec/kmvc.c
> ===================================================================
> --- libavcodec/kmvc.c (revision 12974)
> +++ libavcodec/kmvc.c (working copy)
> @@ -410,5 +410,6 @@
> decode_init,
> NULL,
> decode_end,
> - decode_frame
> + decode_frame,
> + .long_name = "Karl Morton's Video Codec",
> };
ok
[...]
> Index: libavcodec/ljpegenc.c
> ===================================================================
> --- libavcodec/ljpegenc.c (revision 12974)
> +++ libavcodec/ljpegenc.c (working copy)
> @@ -194,4 +194,5 @@
> MPV_encode_init,
> encode_picture_lossless,
> MPV_encode_end,
> + .long_name = "lossless JPEG",
> };
why lowercase?
> Index: libavcodec/loco.c
> ===================================================================
> --- libavcodec/loco.c (revision 12974)
> +++ libavcodec/loco.c (working copy)
> @@ -282,4 +282,5 @@
> NULL,
> decode_frame,
> CODEC_CAP_DR1,
> + .long_name = "LOCO",
> };
hmm, ok
[...]
> Index: libavcodec/mpc7.c
> ===================================================================
> --- libavcodec/mpc7.c (revision 12974)
> +++ libavcodec/mpc7.c (working copy)
> @@ -273,4 +273,5 @@
> NULL,
> mpc7_decode_frame,
> .flush = mpc7_decode_flush,
> + .long_name = "Musepack SV7",
> };
> Index: libavcodec/mpc8.c
> ===================================================================
> --- libavcodec/mpc8.c (revision 12974)
> +++ libavcodec/mpc8.c (working copy)
> @@ -361,4 +361,5 @@
> NULL,
> NULL,
> mpc8_decode_frame,
> + .long_name = "Musepack SV8",
> };
both ok
[...]
> Index: libavcodec/qdrw.c
> ===================================================================
> --- libavcodec/qdrw.c (revision 12974)
> +++ libavcodec/qdrw.c (working copy)
> @@ -151,4 +151,5 @@
> NULL,
> decode_frame,
> CODEC_CAP_DR1,
> + .long_name = "QuickDraw",
> };
Apple QuickDraw, I think
> Index: libavcodec/qpeg.c
> ===================================================================
> --- libavcodec/qpeg.c (revision 12974)
> +++ libavcodec/qpeg.c (working copy)
> @@ -316,4 +316,5 @@
> decode_end,
> decode_frame,
> CODEC_CAP_DR1,
> + .long_name = "Q-team QPEG",
> };
ok
[...]
> Index: libavcodec/rv30.c
> ===================================================================
> --- libavcodec/rv30.c (revision 12974)
> +++ libavcodec/rv30.c (working copy)
> @@ -144,4 +144,5 @@
> ff_rv34_decode_end,
> ff_rv34_decode_frame,
> CODEC_CAP_DR1 | CODEC_CAP_DELAY,
> + .long_name = "RV30 (Real Video version 30)",
> };
> Index: libavcodec/rv40.c
> ===================================================================
> --- libavcodec/rv40.c (revision 12974)
> +++ libavcodec/rv40.c (working copy)
> @@ -276,4 +276,5 @@
> ff_rv34_decode_end,
> ff_rv34_decode_frame,
> CODEC_CAP_DR1 | CODEC_CAP_DELAY,
> + .long_name = "RV40 (Real Video version 40)",
> };
ok as long as all RV decoder are longnamed consistently
[...]
> Index: libavcodec/smacker.c
> ===================================================================
> --- libavcodec/smacker.c (revision 12974)
> +++ libavcodec/smacker.c (working copy)
> @@ -697,7 +697,8 @@
> decode_init,
> NULL,
> decode_end,
> - decode_frame
> + decode_frame,
> + .long_name = "Smacker Video",
> };
>
> AVCodec smackaud_decoder = {
> @@ -708,6 +709,7 @@
> smka_decode_init,
> NULL,
> NULL,
> - smka_decode_frame
> + smka_decode_frame,
> + .long_name = "Smacker Audio",
> };
ok
> Index: libavcodec/targa.c
> ===================================================================
> --- libavcodec/targa.c (revision 12974)
> +++ libavcodec/targa.c (working copy)
> @@ -249,5 +249,6 @@
> targa_end,
> decode_frame,
> 0,
> - NULL
> + NULL,
> + .long_name = "Targa (.tga) image",
> };
Truevision Targa
> Index: libavcodec/tiff.c
> ===================================================================
> --- libavcodec/tiff.c (revision 12974)
> +++ libavcodec/tiff.c (working copy)
> @@ -495,5 +495,6 @@
> tiff_end,
> decode_frame,
> 0,
> - NULL
> + NULL,
> + .long_name = "TIFF image",
> };
ok
> Index: libavcodec/truemotion2.c
> ===================================================================
> --- libavcodec/truemotion2.c (revision 12974)
> +++ libavcodec/truemotion2.c (working copy)
> @@ -887,4 +887,5 @@
> decode_end,
> decode_frame,
> CODEC_CAP_DR1,
> + .long_name = "Duck/ON2 TrueMotion 2",
> };
ok
> Index: libavcodec/truespeech.c
> ===================================================================
> --- libavcodec/truespeech.c (revision 12974)
> +++ libavcodec/truespeech.c (working copy)
> @@ -380,4 +380,5 @@
> NULL,
> NULL,
> truespeech_decode_frame,
> + .long_name = "DSP Group TrueSpeech",
> };
ok
> Index: libavcodec/tscc.c
> ===================================================================
> --- libavcodec/tscc.c (revision 12974)
> +++ libavcodec/tscc.c (working copy)
> @@ -341,5 +341,6 @@
> decode_end,
> decode_frame,
> CODEC_CAP_DR1,
> + .long_name = "TechSmith Camtasia",
> };
TechSmith Screen Capture Codec
> Index: libavcodec/ulti.c
> ===================================================================
> --- libavcodec/ulti.c (revision 12974)
> +++ libavcodec/ulti.c (working copy)
> @@ -410,6 +410,7 @@
> ulti_decode_end,
> ulti_decode_frame,
> CODEC_CAP_DR1,
> - NULL
> + NULL,
> + .long_name = "IBM Ultimotion Video",
> };
IBM UltiMotion (without video and with camelcasing)
> Index: libavcodec/vb.c
> ===================================================================
> --- libavcodec/vb.c (revision 12974)
> +++ libavcodec/vb.c (working copy)
> @@ -277,6 +277,7 @@
> decode_init,
> NULL,
> decode_end,
> - decode_frame
> + decode_frame,
> + .long_name = "Beam Software VB",
> };
ok
> Index: libavcodec/vc1.c
> ===================================================================
> --- libavcodec/vc1.c (revision 12974)
> +++ libavcodec/vc1.c (working copy)
> @@ -4136,7 +4136,8 @@
> vc1_decode_end,
> vc1_decode_frame,
> CODEC_CAP_DELAY,
> - NULL
> + NULL,
> + .long_name = "VC-1",
> };
>
> AVCodec wmv3_decoder = {
> @@ -4149,5 +4150,6 @@
> vc1_decode_end,
> vc1_decode_frame,
> CODEC_CAP_DELAY,
> - NULL
> + NULL,
> + .long_name = "WMV3 (Windows Media Video 9)",
> };
SMPTE VC-1 would be more correct, WMV3 ok if it is consistent with other WMV longnames
> Index: libavcodec/vmnc.c
> ===================================================================
> --- libavcodec/vmnc.c (revision 12974)
> +++ libavcodec/vmnc.c (working copy)
> @@ -517,6 +517,7 @@
> decode_init,
> NULL,
> decode_end,
> - decode_frame
> + decode_frame,
> + .long_name = "VMware Screen Codec (VMnc)",
> };
Fourcc again
> Index: libavcodec/wavpack.c
> ===================================================================
> --- libavcodec/wavpack.c (revision 12974)
> +++ libavcodec/wavpack.c (working copy)
> @@ -582,4 +582,5 @@
> NULL,
> NULL,
> wavpack_decode_frame,
> + .long_name = "WavPack",
> };
ok
> Index: libavcodec/wnv1.c
> ===================================================================
> --- libavcodec/wnv1.c (revision 12974)
> +++ libavcodec/wnv1.c (working copy)
> @@ -141,4 +141,5 @@
> NULL,
> decode_frame,
> CODEC_CAP_DR1,
> + .long_name = "Winnov WNV1",
> };
ok
> Index: libavcodec/ws-snd1.c
> ===================================================================
> --- libavcodec/ws-snd1.c (revision 12974)
> +++ libavcodec/ws-snd1.c (working copy)
> @@ -149,4 +149,5 @@
> NULL,
> NULL,
> ws_snd_decode_frame,
> + .long_name = "Westwood SND1",
> };
Westwood Audio (SND1) seems to be better
> Index: libavcodec/xl.c
> ===================================================================
> --- libavcodec/xl.c (revision 12974)
> +++ libavcodec/xl.c (working copy)
> @@ -135,4 +135,5 @@
> NULL,
> decode_frame,
> CODEC_CAP_DR1,
> + .long_name = "Miro VideoXL",
> };
ok
> Index: libavcodec/zmbv.c
> ===================================================================
> --- libavcodec/zmbv.c (revision 12974)
> +++ libavcodec/zmbv.c (working copy)
> @@ -661,6 +661,7 @@
> decode_init,
> NULL,
> decode_end,
> - decode_frame
> + decode_frame,
> + .long_name = "Zip Motion Blocks Video (ZMBV)",
> };
>
> Index: libavcodec/zmbvenc.c
> ===================================================================
> --- libavcodec/zmbvenc.c (revision 12974)
> +++ libavcodec/zmbvenc.c (working copy)
> @@ -319,4 +319,5 @@
> encode_frame,
> encode_end,
> .pix_fmts = (enum PixelFormat[]){PIX_FMT_PAL8, -1},
> + .long_name = "Zip Motion Blocks Video (ZMBV)",
> };
ok
More information about the ffmpeg-devel
mailing list