[FFmpeg-devel] On need in AVFrame accessors

Michael Niedermayer michaelni at gmx.at
Wed Aug 22 18:19:35 CEST 2012

On Wed, Aug 22, 2012 at 03:33:49PM +0300, Andrey Utkin wrote:
> 2012/8/22 Nicolas George <nicolas.george at normalesup.org>:
> > The direct get/set operations are forbidden
> Having looked at AVFrame documentation comment, i've understood the
> situation about it (although i don't like it).
> But i was intrigued initially by accessor to
> AVCodecContext.pkt_timebase; AVCodecContext description does not
> contain same warning as AVFrame.
> Of course, it is not so important issue, and thanks to Nicolas for
> explanation, but i'm happy i have worded my opinion. And my opinion is
> that complication is creeping :)
> P.S. I would not open such topic if not necessity to copy&paste
> function avpriv_set_pts_info() into muxer, which i want to compile out
> of ffmpeg tree. Looking at avpriv_set_pts_info() code i have
> discovered this new thing. Maybe we should make such function public
> (and patch is welcome)? Or building muxers out of tree is not
> supported? I remember i have built filters out of tree finely.

IMHO set_pts_info() should not be private

Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Breaking DRM is a little like attempting to break through a door even
though the window is wide open and the only thing in the house is a bunch
of things you dont want and which you would get tomorrow for free anyway
-------------- 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/20120822/998e3c1c/attachment.asc>

More information about the ffmpeg-devel mailing list