[Speex-dev] GCC/ELF Visibility patch (fwd)

[Thorvald Natvig]

I just tried, and Win32 support will require adding __declspec to all 
the header files as well, or you will get redifinition errors. 
Alternately you can just add
#define EXPORT
to win32/config.h; this will give the same exact result you had before 
the patch.

Jean-Marc Valin wrote:
> Alexander, what do you think about that when it comes to Win32 support?
>
> 	Jean-Marc
>
> Thorvald Natvig a écrit :
>   
>> Hi,
>>
>> I've attached a patch against SVN r14645 which adds GCC visibility
>> information to all symbols exported from libspeex.so and libspeexdsp.so.
>> It includes a configure.ac change to test that both the compiler flags
>> and __attribute__((visibility)) works, and if so will
>> #define EXPORT __attribute__((visibility("default")))
>> and if not
>> #define EXPORT
>>
>> I've attached a diff output of "nm -C -D" for both libraries (compiled
>> on x86_64 Linux), and verified that none of the symbols removed are
>> mentioned in any header file under include/
>>
>> If this gets accepted, I'll make a similar effort for Win32 (where it
>> hopefully should suffice to #define EXPORT in the config.h)
>>
>>
>> ------------------------------------------------------------------------
>>
>> _______________________________________________
>> Speex-dev mailing list
>> Speex-dev at xiph.org
>> http://lists.xiph.org/mailman/listinfo/speex-dev
>>