[FFmpeg-devel] [PATCH 6/9] lavfi: move AVFilterLink declaration to its own header.

Michael Niedermayer michael at niedermayer.cc
Tue Dec 27 22:06:52 EET 2016


On Tue, Dec 27, 2016 at 07:02:07PM +0100, Nicolas George wrote:
> AVFilterLink contains fields of internal types that are better
> isolated. Furthermore, with the prospect of inter-filters
> threading, it is better if filters do not access links directly.
> 
> Signed-off-by: Nicolas George <george at nsup.org>
> ---
>  libavfilter/Makefile       |   1 +
>  libavfilter/avfilter.h     | 217 +--------------------------------------
>  libavfilter/avfilterlink.h | 248 +++++++++++++++++++++++++++++++++++++++++++++
>  3 files changed, 250 insertions(+), 216 deletions(-)
>  create mode 100644 libavfilter/avfilterlink.h
> 
> 
> That way, making it public or private is just a matter of changing one line
> in the Makefile.

LGTM

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Complexity theory is the science of finding the exact solution to an
approximation. Benchmarking OTOH is finding an approximation of the exact
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20161227/f170b48c/attachment.sig>


More information about the ffmpeg-devel mailing list