[FFmpeg-devel] Document AVOption struct
Mon Jun 16 10:25:12 CEST 2008
On date Monday 2008-06-16 04:45:50 +0200, Michael Niedermayer encoded:
> On Sun, Jun 15, 2008 at 07:21:59PM -0400, The Wanderer wrote:
> > Stefano Sabatini wrote:
> > > On date Saturday 2008-06-14 22:47:33 +0200, Michael Niedermayer
> > > encoded:
> > >
> > >> On Sat, Jun 14, 2008 at 01:34:04AM +0200, Stefano Sabatini wrote:
> > >>> I tried to improve it, I think now it's simpler and clearer:
> > >>> /**
> > >>> * Aggregates options into a single logical unit. Named constants
> > >>> * belonging to the same option share the same unit, which
> > >>> * corresponds to the name of that option.
> > >>> */
> > >>
> > >> A function does something a variables is something.
> > >>
> > >> A description of a variable should not be about how it is used but
> > >> about what it is primarely. After one has defined what the variable
> > >> is/contains one can, if needed explain what that is used for if
> > >> thats is important.
> > >
> > > Got it, please check if it is better now.
> > For as much as my two cents are worth, looks good to me.
> I agree from a english language point of view ...
> sadly it contains a bug now ;)
> the unit is not neccessarily the name of the option it belongs to, as example
> see cmp_func. The constants "belong" to several options (cmp,subcmp,...)
Doh!! Corrected here, also fixed the "default/the default", issue.
FFmpeg = Funny Freak Maxi Portable EntanGlement
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 1962 bytes
Desc: not available
More information about the ffmpeg-devel