[FFmpeg-devel] [PATCH 1/3] lavu/opt: document underlying C types for enum AVOptionType

[Michael Niedermayer]

On Thu, Aug 22, 2024 at 02:39:10PM +0200, Anton Khirnov wrote:
> Quoting Michael Niedermayer (2024-08-20 22:57:37)
> > Hi
> > 
> > On Sun, Aug 18, 2024 at 01:26:53PM +0200, Anton Khirnov wrote:
> > > ---
> > >  libavutil/opt.h | 78 +++++++++++++++++++++++++++++++++++++++++++++++--
> > >  1 file changed, 75 insertions(+), 3 deletions(-)
> > > 
> > > diff --git a/libavutil/opt.h b/libavutil/opt.h
> > > index 07e27a9208..23bc495158 100644
> > > --- a/libavutil/opt.h
> > > +++ b/libavutil/opt.h
> > > @@ -240,26 +240,98 @@
> > >   * before the file is actually opened.
> > >   */
> > >  
> > > +/**
> > > + * An option type determines:
> > > + * - for native access, the underlying C type of the field that an AVOption
> > > + *   refers to;
> > > + * - for foreign access, the semantics of accessing the option through this API,
> > > + *   e.g. which av_opt_get_*() and av_opt_set_*() functions can be called, or
> > > + *   what format will av_opt_get()/av_opt_set() expect/produce.
> > 
> > "foreign access" ? is this a standard/ common term ?
> > if not, maybe somone has a more descriptive more common term/idea
> 
> The term is defined higher up in the file, cf. fc706276c05

if it sounds fine to you, then no objection.
I wonder if direct vs access through accessor functions are clearer
terms though

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The misfortune of the wise is better than the prosperity of the fool.
-- Epicurus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20240823/d9abe8a7/attachment.sig>