From 14c3ce5737e17671045d70f4eff3bd0fca91ad99 Mon Sep 17 00:00:00 2001 From: Tsuda Kageyu Date: Tue, 20 Dec 2016 11:54:06 +0900 Subject: [PATCH] Remove all the optional dependencies on Boost. --- ConfigureChecks.cmake | 68 +++++++++++++++++------------------------ config.h.cmake | 1 - taglib/toolkit/tutils.h | 22 +++---------- 3 files changed, 32 insertions(+), 59 deletions(-) diff --git a/ConfigureChecks.cmake b/ConfigureChecks.cmake index 98751b06..f69b4380 100644 --- a/ConfigureChecks.cmake +++ b/ConfigureChecks.cmake @@ -103,69 +103,57 @@ endif() # Determine which kind of byte swap functions your compiler supports. check_cxx_source_compiles(" - #include int main() { - boost::endian::endian_reverse(static_cast(1)); - boost::endian::endian_reverse(static_cast(1)); - boost::endian::endian_reverse(static_cast(1)); + __builtin_bswap16(0); + __builtin_bswap32(0); + __builtin_bswap64(0); return 0; } -" HAVE_BOOST_BYTESWAP) +" HAVE_GCC_BYTESWAP) -if(NOT HAVE_BOOST_BYTESWAP) +if(NOT HAVE_GCC_BYTESWAP) check_cxx_source_compiles(" + #include int main() { - __builtin_bswap16(0); - __builtin_bswap32(0); - __builtin_bswap64(0); + __bswap_16(0); + __bswap_32(0); + __bswap_64(0); return 0; } - " HAVE_GCC_BYTESWAP) + " HAVE_GLIBC_BYTESWAP) - if(NOT HAVE_GCC_BYTESWAP) + if(NOT HAVE_GLIBC_BYTESWAP) check_cxx_source_compiles(" - #include + #include int main() { - __bswap_16(0); - __bswap_32(0); - __bswap_64(0); + _byteswap_ushort(0); + _byteswap_ulong(0); + _byteswap_uint64(0); return 0; } - " HAVE_GLIBC_BYTESWAP) + " HAVE_MSC_BYTESWAP) - if(NOT HAVE_GLIBC_BYTESWAP) + if(NOT HAVE_MSC_BYTESWAP) check_cxx_source_compiles(" - #include + #include int main() { - _byteswap_ushort(0); - _byteswap_ulong(0); - _byteswap_uint64(0); + OSSwapInt16(0); + OSSwapInt32(0); + OSSwapInt64(0); return 0; } - " HAVE_MSC_BYTESWAP) + " HAVE_MAC_BYTESWAP) - if(NOT HAVE_MSC_BYTESWAP) + if(NOT HAVE_MAC_BYTESWAP) check_cxx_source_compiles(" - #include + #include int main() { - OSSwapInt16(0); - OSSwapInt32(0); - OSSwapInt64(0); + swap16(0); + swap32(0); + swap64(0); return 0; } - " HAVE_MAC_BYTESWAP) - - if(NOT HAVE_MAC_BYTESWAP) - check_cxx_source_compiles(" - #include - int main() { - swap16(0); - swap32(0); - swap64(0); - return 0; - } - " HAVE_OPENBSD_BYTESWAP) - endif() + " HAVE_OPENBSD_BYTESWAP) endif() endif() endif() diff --git a/config.h.cmake b/config.h.cmake index fd716134..c1a31094 100644 --- a/config.h.cmake +++ b/config.h.cmake @@ -4,7 +4,6 @@ #define TAGLIB_CONFIG_H /* Defined if your compiler supports some byte swap functions */ -#cmakedefine HAVE_BOOST_BYTESWAP 1 #cmakedefine HAVE_GCC_BYTESWAP 1 #cmakedefine HAVE_GLIBC_BYTESWAP 1 #cmakedefine HAVE_MSC_BYTESWAP 1 diff --git a/taglib/toolkit/tutils.h b/taglib/toolkit/tutils.h index b114619e..6d96cd12 100644 --- a/taglib/toolkit/tutils.h +++ b/taglib/toolkit/tutils.h @@ -34,9 +34,7 @@ # include #endif -#if defined(HAVE_BOOST_BYTESWAP) -# include -#elif defined(HAVE_MSC_BYTESWAP) +#if defined(HAVE_MSC_BYTESWAP) # include #elif defined(HAVE_GLIBC_BYTESWAP) # include @@ -63,11 +61,7 @@ namespace TagLib */ inline unsigned short byteSwap(unsigned short x) { -#if defined(HAVE_BOOST_BYTESWAP) - - return boost::endian::endian_reverse(static_cast(x)); - -#elif defined(HAVE_GCC_BYTESWAP) +#if defined(HAVE_GCC_BYTESWAP) return __builtin_bswap16(x); @@ -99,11 +93,7 @@ namespace TagLib */ inline unsigned int byteSwap(unsigned int x) { -#if defined(HAVE_BOOST_BYTESWAP) - - return boost::endian::endian_reverse(static_cast(x)); - -#elif defined(HAVE_GCC_BYTESWAP) +#if defined(HAVE_GCC_BYTESWAP) return __builtin_bswap32(x); @@ -138,11 +128,7 @@ namespace TagLib */ inline unsigned long long byteSwap(unsigned long long x) { -#if defined(HAVE_BOOST_BYTESWAP) - - return boost::endian::endian_reverse(static_cast(x)); - -#elif defined(HAVE_GCC_BYTESWAP) +#if defined(HAVE_GCC_BYTESWAP) return __builtin_bswap64(x); -- 2.40.0