[FFmpeg-devel] [PATCH] Fill unspecified AVCodec long_name fields
Stefano Sabatini
stefano.sabatini-lala
Sat Apr 26 11:51:52 CEST 2008
On date Saturday 2008-04-26 07:31:17 +0300, Kostya encoded:
> 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 ;)
[...]
> > 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"
What about:
Autodesk Animator Flic (.fli/.flc) Video?
or simply
Autodesk Animator Flic Video?
> [...]
>
> > 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"?
ok
> [...]
> > 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
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?
changed to -> "Lossless JPEG"
>
> > 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
ok
> > 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
ok
> > 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)
ok
> > 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
WMV3 -> Windows Media Video 9, also I changed accordingly the other
WMVX long names.
Another solution could be:
.long_name = "SMPTE VC-1 Simple-Main Profile / Windows Media Video 9",
> > 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
ok
> > 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
ok
> > 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
Thanks for your review!
New patch attached.
--
FFmpeg = Faboulous Foolish MultiPurpose EnGine
-------------- next part --------------
A non-text attachment was scrubbed...
Name: add-missing-long-names-02.patch
Type: text/x-diff
Size: 62157 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080426/ed4dc5fb/attachment.patch>
More information about the ffmpeg-devel
mailing list