[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