[Ffmpeg-devel] [PATCH] AVOptions description sync

Guillaume POIRIER poirierg
Mon Sep 11 16:01:47 CEST 2006


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.

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