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

Kostya kostya.shishkov
Sat Apr 26 12:11:23 CEST 2008


On Sat, Apr 26, 2008 at 11:51:52AM +0200, Stefano Sabatini wrote:
> 
> 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 ;)
 
[...]

> Thanks for your review!
> 
> New patch attached.

Either wait for it to be approved by others or ask patch monkeys to apply
ook'ed parts. Or maybe I will do it (the parts for codecs maintained by me).




More information about the ffmpeg-devel mailing list