[FFmpeg-devel] [PATCH 1/3] lavc: implement accessors for some AVFrame fields.
michaelni at gmx.at
Sun Apr 29 14:47:47 CEST 2012
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.
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
There seems to be only one solution to NIH syndrom, ... a shooting squad
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 198 bytes
Desc: Digital signature
More information about the ffmpeg-devel