[FFmpeg-devel] [PATCH 1/3] lavc: implement accessors for some AVFrame fields.
ubitux at gmail.com
Sun Apr 29 17:57:05 CEST 2012
On Sun, Apr 29, 2012 at 02:47:47PM +0200, Michael Niedermayer wrote:
> On Sun, Apr 29, 2012 at 12:10:07PM +0200, Clément Bœsch wrote:
> > On Sun, Apr 29, 2012 at 11:21:44AM +0200, Nicolas George wrote:
> > > Compared to av_opt_ptr, accessors bring:
> > >
> > > - better performance (negligible);
> > > - compile-time type check;
> > > - link-time existence check
> > > (or at worst, a dynamic linker error instead of a NULL dereference).
> > >
> > I like the idea, but by the way, should we make this for all the
> > user-editable/readable fields, with a doxy associated with them?
> > "sometimes there is a getter/setter, sometimes not; how am I supposed to
> > know if I can change this given field?"
> all fields are supposed to list this already:
> > > * - encoding: unused
> > > * - decoding: read by user.
> for example
Yes but adding a bunch of getter/setter for a limited range of options
might be confusing since the API gets inconsistent. Of course it's somehow
already the case, but in the current state it appears like what it really
is: a workaround for ABI issues. After the patch it looks more like
helpers for users. That's the reason I was just raising the idea of
extending this to every user configurable fields.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 490 bytes
Desc: not available
More information about the ffmpeg-devel