[FFmpeg-devel] [PATCH 1/3] lavc: implement accessors for some AVFrame fields.
nicolas.george at normalesup.org
Wed May 2 12:04:31 CEST 2012
Le quartidi 14 floréal, an CCXX, Stefano Sabatini a écrit :
> > +MAKE_ACCESSORS(AVFrame, frame, int64_t, best_effort_timestamp)
> > +MAKE_ACCESSORS(AVFrame, frame, int64_t, pkt_pos)
> > +MAKE_ACCESSORS(AVFrame, frame, int64_t, channel_layout)
> > +MAKE_ACCESSORS(AVFrame, frame, int, sample_rate)
> What about to move this to the header? This way you avoid the IMO
> redundant declarations.
You mean as inline functions? Unfortunately, that can not work: an inline
function in a header behaves exactly like code in the client program, i.e.,
accessing directly the fields. And that is precisely what we want to avoid.
Additionally, I like having the prototypes completely written in the header,
as it is self-documenting. Otherwise, a macro could have served to build
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 198 bytes
Desc: Digital signature
More information about the ffmpeg-devel