On 12.07.2014 15:40, Ronald S. Bultje wrote:
Hi,
On Sat, Jul 12, 2014 at 9:35 AM, Andreas Cadhalpun <
andreas.cadhal...@googlemail.com> wrote:
Hi,
I'm a bit puzzled by this commit.
On 30.06.2014 01:19, James Almer wrote:
From: Ronald S. Bultje <rsbul...@gmail.com>
---
libswresample/resample_template.c | 23 +---
libswresample/x86/resample.asm | 225
++++++++++++++++++++++++++---------
libswresample/x86/resample_mmx.h | 110 -----------------
libswresample/x86/resample_x86_dsp.c | 44 +++----
4 files changed, 187 insertions(+), 215 deletions(-)
[...]
diff --git a/libswresample/x86/resample_mmx.h
b/libswresample/x86/resample_mmx.h
index 94237b0..b0ea496 100644
--- a/libswresample/x86/resample_mmx.h
+++ b/libswresample/x86/resample_mmx.h
@@ -22,116 +22,6 @@
#include "libavutil/cpu.h"
#include "libswresample/swresample_internal.h"
-DECLARE_ALIGNED(16, const uint64_t, ff_resample_int16_rounder)[2] = {
0x0000000000004000ULL, 0x0000000000000000ULL};
This removes the symbol ff_resample_int16_rounder from libswresample,
which is a backwards incompatible ABI change.
So can someone explain to me, why this hasn't been accompanied by a major
SOVERSION bump of libswresample?
All ff_* symbols are internal.
Then shouldn't all of them be declared with something like
'__attribute__ ((visibility ("hidden")))'?
Best regards,
Andreas
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
