[FFmpeg-devel] [PATCH 1/3] lavc: implement accessors for some AVFrame fields.

Michael Niedermayer 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.
for example

[...]

-- 
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
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120429/40086587/attachment.asc>


More information about the ffmpeg-devel mailing list