On Mon, Jul 06, 2015 at 12:13:41PM +0200, Hendrik Leppkes wrote: > On Mon, Jul 6, 2015 at 12:07 PM, Michael Niedermayer <michae...@gmx.at> wrote: > > Signed-off-by: Michael Niedermayer <michae...@gmx.at> > > --- > > libavcodec/internal.h | 3 +++ > > 1 file changed, 3 insertions(+) > > > > diff --git a/libavcodec/internal.h b/libavcodec/internal.h > > index 2feb68b..c410f8b 100644 > > --- a/libavcodec/internal.h > > +++ b/libavcodec/internal.h > > @@ -210,6 +210,9 @@ int avpriv_unlock_avformat(void); > > * or allocated in this function. > > * > > * @param avctx the AVCodecContext of the encoder > > + * this can be set to NULL to avoid the use of > > internal->byte_buffer > > + * in the allocation algorithm. When the final used size is > > well > > + * known it is more efficient to not use > > internal->byte_buffer. > > * @param avpkt the AVPacket > > * If avpkt->data is already set, avpkt->size is checked > > * to ensure it is large enough. > > This seems terrible. Can't it somehow be made smarter? > Passing a NULL context is hardly intuitive or obvious.
i can add a flag to the function arguments if you think thats better? [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Opposition brings concord. Out of discord comes the fairest harmony. -- Heraclitus
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
