[FFmpeg-devel] [PATCH 1/2] avfilter: change ff_inlink_make_frame_writable() to take AVFrame* argument

wm4 nfxjfg at googlemail.com
Sun Jan 29 14:21:30 EET 2017


On Sun, 29 Jan 2017 13:06:13 +0100
Nicolas George <george at nsup.org> wrote:

> Le decadi 10 pluviôse, an CCXXV, wm4 a écrit :
> > Then  you should stop moving away from it. The av_frame_make_writable
> > design is superior for multiple reasons, and was intentionally designed
> > this way by an intelligent mind with a lot of API-foresight.
> > 
> > If you "move away from it" just like this, you create a major
> > inconsistency as well.
> > 
> > I haven't heard a good argument as to why its API is supposed to be
> > better than av_frame_make_writable.
> > 
> > I one the other hand delivered a bunch of arguments to which you didn't
> > reply.  
> 
> Rude again.

Anyway, since you're unwilling to discuss the technical points of this
patch, and seeing you're not maintainer of libavfilter or this file, I
will push patch 1/2 tomorrow.


More information about the ffmpeg-devel mailing list