[Ffmpeg-devel] [PATCH] AVOptions description sync

Guillaume POIRIER poirierg
Wed Sep 13 11:04:01 CEST 2006


Hi,

On 9/11/06, Guillaume POIRIER <poirierg at gmail.com> wrote:
> Hi,
>
> On 9/11/06, Panagiotis Issaris <takis.issaris at uhasselt.be> wrote:
> > Hi,
> >
> > The attached patch adds descriptions to the libavcodec AVOptions array, taken from the
> > OptionDef options[] array.
> >
> >  utils.c |   78 ++++++++++++++++++++++++++++++++--------------------------------
> >  1 file changed, 39 insertions(+), 39 deletions(-)
> >
> > A few descriptions had to be altered, such the video bitrate, as the AVOption parameter
> > wants the value specified to be the actual _bits_ per second, while the OptionDef parameter
> > takes Kbits/s. To keep this patch from doing more then one thing, I did not address (or
> > even tried to address) this issue. But at least the patch does not modify any behavior,
> > except for showing more (and correct) info.
>
> Patch looks good to me, and the information look also correct.
> It also looks simple and uncontroversial
>
> Will apply late this afternoon unless someone disagrees.

Finally applied.

Guillaume
-- 
With DADVSI (http://en.wikipedia.org/wiki/DADVSI), France finally has
a lead on USA on selling out individuals right to corporations!
Vive la France!




More information about the ffmpeg-devel mailing list