search for: flac__use_visibility_attr

...@@ #define FLAC_API __declspec(dllimport) #endif +#elif defined(__OS2__) +#if defined(FLAC_API_EXPORTS) && defined(__WATCOMC__) && defined(__SW_BD) +#define FLAC_API __declspec(__cdecl) __declspec(dllexport) +#else +#define FLAC_API __declspec(dllexport) +#endif + #elif defined(FLAC__USE_VISIBILITY_ATTR) #define FLAC_API __attribute__ ((visibility ("default"))) Another option is for the caller to define FLAC_API, eg #define FLAC_API __declspec(__cdecl) #include <flac\all.h> diff --git a/include/FLAC/export.h b/include/FLAC/export.h index d52f0bb..07cfe59 100644 --- a/include/FLA...

...ine FLAC_API -#elif defined(_MSC_VER) +#elif defined(_WIN32) /*defined(_MSC_VER)*/ #ifdef FLAC_API_EXPORTS -#define FLAC_API _declspec(dllexport) +#define FLAC_API __declspec(dllexport) #else -#define FLAC_API _declspec(dllimport) +#define FLAC_API __declspec(dllimport) #endif #elif defined(FLAC__USE_VISIBILITY_ATTR) diff --git a/include/FLAC++/export.h b/include/FLAC++/export.h index 11c09e6..ab9b439 100644 --- a/include/FLAC++/export.h +++ b/include/FLAC++/export.h @@ -59,11 +59,11 @@ #if defined(FLAC__NO_DLL) #define FLACPP_API -#elif defined(_MSC_VER) +#elif defined(_WIN32) /*defined(_MSC_VER)*/ #ifde...

...est x$enable_gcc_werror = "xyes" ; then CFLAGS="-Wall -Wextra -Werror $CFLAGS" CXXFLAGS="-Wall -Wextra -Werror $CXXFLAGS" - fi fi + if test "$GCC_MAJOR_VERSION" -ge 4 && test "$OBJ_FORMAT" = elf; then + CPPFLAGS="$CPPFLAGS -DFLAC__USE_VISIBILITY_ATTR" + CFLAGS="$CFLAGS -fvisibility=hidden" + CXXFLAGS="$CXXFLAGS -fvisibility=hidden" + fi +fi + #@@@ AM_CONDITIONAL(FLaC__HAS_AS__TEMPORARILY_DISABLED, test "yes" = "no") diff --git a/include/FLAC++/export.h b/include/FLAC++/export.h index e3bc51f..d...

On 01/22/17 02:00 PM, Ozkan Sezer wrote: > Question: Does emx support __declspec(dllexport) so I can adjust > these changes? Because the emx build of the dll seems to have > exported_everything_ :( GCC supports __declspec(dllexport) though it still needs a def file, with no exports. Libtool doesn't currently and as flac uses libtool... Dave

The attached set of patches adds support for OS/2 using Watcom compiler (tested with Open Watcom 1.9). My only interest was building a working dll (the last patch in the set adds a makefile for it), therefore I did not touch other places: If there is interest, I can do so. Regards. -- O.S. -------------- next part -------------- A non-text attachment was scrubbed... Name:

....dll link with Watcom with something like >>> diff --git a/include/FLAC/export.h b/include/FLAC/export.h >>> index d52f0bb..96d5422 100644 >>> --- a/include/FLAC/export.h >>> +++ b/include/FLAC/export.h >>> @@ -69,6 +69,9 @@ >>> #elif defined(FLAC__USE_VISIBILITY_ATTR) >>> #define FLAC_API __attribute__ ((visibility ("default"))) >>> >>> +#elif defined (__WATCOMC__) && defined (__OS2__) >>> +#define FLAC_API __declspec(__cdecl) >>> + >>> #else >>> #define FLAC_API >>...

...llexport) +#if defined(__GNUC__) +#define FLAC_API __attribute__((dllexport)) +#else +#define FLAC_API __declspec(dllexport) +#endif #else -#define FLAC_API _declspec(dllimport) +#if defined(__GNUC__) +#define FLAC_API +#else +#define FLAC_API __declspec(dllimport) +#endif #endif #elif defined(FLAC__USE_VISIBILITY_ATTR) --- flac-1.3.0/include/FLAC++/export.h.orig 2013-05-26 09:30:33.920202000 +0000 +++ flac-1.3.0/include/FLAC++/export.h 2014-05-03 09:12:38.663876500 +0000 @@ -59,11 +59,19 @@ #if defined(FLAC__NO_DLL) #define FLACPP_API -#elif defined(_MSC_VER) -#ifdef FLACPP_API_EXPORTS -#define FLACPP_API _d...

...SC_VER) -#ifdef FLAC_API_EXPORTS -#define FLAC_API _declspec(dllexport) +#elif defined(_WIN32) +#if defined(FLAC_API_EXPORTS) || defined(DLL_EXPORT) +#define FLAC_API __declspec(dllexport) #else -#define FLAC_API _declspec(dllimport) +#define FLAC_API __declspec(dllimport) #endif #elif defined(FLAC__USE_VISIBILITY_ATTR) diff --git a/include/FLAC++/export.h b/include/FLAC++/export.h index 11c09e6..91649a5 100644 --- a/include/FLAC++/export.h +++ b/include/FLAC++/export.h @@ -59,11 +59,11 @@ #if defined(FLAC__NO_DLL) #define FLACPP_API -#elif defined(_MSC_VER) -#ifdef FLACPP_API_EXPORTS -#define FLACPP_API _dec...

They are not needed since the section is defined in nasm.h. --- src/libFLAC/ia32/bitreader_asm.nasm | 4 ---- src/libFLAC/ia32/cpu_asm.nasm | 4 ---- src/libFLAC/ia32/fixed_asm.nasm | 4 ---- src/libFLAC/ia32/lpc_asm.nasm | 4 ---- src/libFLAC/ia32/stream_encoder_asm.nasm | 4 ---- 5 files changed, 20 deletions(-) diff --git

...r defined (__WATCOMC__) && defined (__OS2__) Will the GCC built flac8.dll link with Watcom with something like diff --git a/include/FLAC/export.h b/include/FLAC/export.h index d52f0bb..96d5422 100644 --- a/include/FLAC/export.h +++ b/include/FLAC/export.h @@ -69,6 +69,9 @@ #elif defined(FLAC__USE_VISIBILITY_ATTR) #define FLAC_API __attribute__ ((visibility ("default"))) +#elif defined (__WATCOMC__) && defined (__OS2__) +#define FLAC_API __declspec(__cdecl) + #else #define FLAC_API Not sure if a OMF import lib is required but easy enough to create with implib or emximp Dave