On Sat, Aug 04, 2012 at 03:44:34AM +0100, Mans Rullgard wrote:
> This file defines a single, huge function, MPV_motion(), which
> although being declared inline is not actually inlined by the
> compiler (for good reason).  There is thus no sense in defining
> this function in a header file, resulting in multiple copies of
> it in the final library.
> 
> Signed-off-by: Mans Rullgard <[email protected]>
> ---
>  libavcodec/Makefile                                |  2 +-
>  libavcodec/dnxhdenc.c                              |  1 -
>  libavcodec/mpegvideo.c                             |  5 ++--
>  libavcodec/mpegvideo.h                             | 15 +++++++++++
>  libavcodec/mpegvideo_enc.c                         | 11 ++++----
>  .../{mpegvideo_common.h => mpegvideo_motion.c}     | 30 
> +++++-----------------
>  6 files changed, 30 insertions(+), 34 deletions(-)
>  rename libavcodec/{mpegvideo_common.h => mpegvideo_motion.c} (97%)

LGTM
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to