[FFmpeg-devel] [PATCH] avcodec/idctdsp: make add/put_pixels_clamped_c internal functions

Michael Niedermayer michaelni at gmx.at
Wed Aug 13 01:00:15 CEST 2014


On Tue, Aug 12, 2014 at 06:22:24PM -0300, James Almer wrote:
> This reduces code duplication and differences with the fork.
> 
> Signed-off-by: James Almer <jamrial at gmail.com>
> ---
>  libavcodec/idctdsp.c  | 49 +++++++++++++++++++++++++++++++++++++++++++++----
>  libavcodec/idctdsp.h  | 46 +++++-----------------------------------------
>  libavcodec/xvididct.c |  4 ++--
>  3 files changed, 52 insertions(+), 47 deletions(-)

breaks arm

libavcodec/arm/idctdsp_init_arm.c:33:15: error: ‘ff_put_pixels_clamped’ redeclared as different kind of symbol
libavcodec/idctdsp.h:51:6: note: previous declaration of ‘ff_put_pixels_clamped’ was here
libavcodec/arm/idctdsp_init_arm.c:34:15: error: ‘ff_add_pixels_clamped’ redeclared as different kind of symbol
libavcodec/idctdsp.h:53:6: note: previous declaration of ‘ff_add_pixels_clamped’ was here

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

I do not agree with what you have to say, but I'll defend to the death your
right to say it. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140813/2763d9f4/attachment.asc>


More information about the ffmpeg-devel mailing list