[FFmpeg-devel] [PATCH] Document opt.h:av_find_opt()

Stefano Sabatini stefano.sabatini-lala
Tue Jul 1 01:32:10 CEST 2008


On date Monday 2008-06-30 19:14:24 -0400, The Wanderer encoded:
> Stefano Sabatini wrote:
> 
> > Hi all,
> > continues the opt.h documentation saga.
> > 
> > I took account in this patch to all the previous remarks done by
> > Diego, Michael, The Wanderer as much as my mind made it possible ;-).
> 
> Not bad, but I see a couple of possible improvements. (This is almost on 
> the level of nitpicking, but I points 'em out as I sees 'em...)
> 
> > + * Looks for an option in \p obj. Looks only for the options which
> > + * have the flags set as specified in \p mask and \p flags (that is,
> > + * for which is: opt->flags & mask == flags).
> 
> I would probably say something more like "for which it is the case that"
> (or, less precisely but more simply, just "for which") and drop the
> colon.
> 
> > + * @param[in] obj a pointer to an #AVClass struct or to an #AVClass
> > + * context struct
> > + * @param[in] name the name of the option to look for
> > + * @param[in[ unit the unit of the option to look for or any if NULL
> > + * @return a pointer to the option found or NULL if no option
> > + * has been found
> 
> I would add a comma before the "or" on both of these last two.

Mmh... this still continues to seem strange to me, nonetheless I
applied all your suggestions.

> Other than that, looks fairly good to me.

Thanks for the review, new patch attached.
-- 
FFmpeg = Fantastic Fostering Maxi Pitiless Exploitable Guru
-------------- next part --------------
A non-text attachment was scrubbed...
Name: document-av-opt-find-01.patch
Type: text/x-diff
Size: 961 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080701/407e5341/attachment.patch>



More information about the ffmpeg-devel mailing list