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

[Aurelien Jacobs]

On Sat, 26 Apr 2008 13:00:56 +0200
Stefano Sabatini <stefano.sabatini-lala at poste.it> wrote:

> On date Saturday 2008-04-26 03:41:38 -0600, Loren Merritt encoded:
> > On Sat, 26 Apr 2008, Stefano Sabatini wrote:
> > > On date Friday 2008-04-25 19:11:26 -0400, Alexander Strange encoded:
> > >> On Apr 25, 2008, at 6:24 PM, Stefano Sabatini wrote:
> > >>
> > >>> +    .long_name = "FFVHuff",
> > >>
> > >> I'm not sure that's how it's capitalized, but i might be wrong.
> > >
> > > FFVhuff?
> > >
> > > Also can someone suggest a more meaningful long name?
> > 
> > As the person who named that codec I can say that it wasn't intended to 
> > stand for anything. But if I have to pick a backronym, then
> > FFmpeg's Variant of Huffyuv.
> 
> Thanks, new patch attached.
> 
> [...]
> 
>  /* flash version, not flipped upside-down */
> @@ -651,6 +652,7 @@
>      vp56_free,
>      vp56_decode_frame,
>      CODEC_CAP_DR1,
> +    .long_name = "On2 VP6 flash version, not flipped upside-down",
>  };

Long name is nice, but maybe this one is overly long.
I think this would be enough:
  "On2 VP6 (flash version)"

>  /* flash version, not flipped upside-down, with alpha channel */
> @@ -664,4 +666,5 @@
>      vp56_free,
>      vp56_decode_frame,
>      CODEC_CAP_DR1,
> +    .long_name = "On2 VP6 flash version, not flipped upside-down, with alpha channel",
>  };

  "On2 VP6 (flash version, with alpha channel)"

Aurel