[FFmpeg-devel] [PATCH 12/17] lavfi: move ff_update_link_current_pts() into the utility functions.
michael at niedermayer.cc
Sat Dec 31 18:02:17 EET 2016
On Sat, Dec 31, 2016 at 02:29:45PM +0100, Nicolas George wrote:
> Le primidi 11 nivôse, an CCXXV, Michael Niedermayer a écrit :
> > slightly off topic but ff_update_link_current_pts() has no
> > documentation in the header
> I also noticed it. Also, it is now only used in avfilter.c, so it could
> be made static. I will have to address it when I get to maintaining a
> priority queue of filters that are ready and deprecating
> avfilter_graph_request_oldest(), since they are related
> (ff_update_link_current_pts() maintains a heap of sink links by
> > also the commit message is a bit terse, why is it moved
> > does this add or remove calls to it in some codepathes ?
> I reworded it as:
> lavfi: move ff_update_link_current_pts() into the utility functions.
> It does not change anything for the existing filters and makes
> better code fatrorization when future code will use the utility
> My reason to have it in a separate commit is to be able to run FATE
> before and after the change to check it did not break anything.
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
There will always be a question for which you do not know the correct answer.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 181 bytes
Desc: Digital signature
More information about the ffmpeg-devel