Clean up int128_t macro gating

Summary: There is inconsistent use of `#ifdef` and `#if`. Cleaning this up

Reviewed By: yfeldblum

Differential Revision: D3523561

fbshipit-source-id: 23f62d632efeb1b9b137b3e3582c37aae591d901

diff --git a/folly/Conv.cpp b/folly/Conv.cpp
index 199819e3d1860df93434a4dab0fe6b3c89d8a2c5..e11d8693a4ff95540bb2f3c30be8ff3cf861e3fb 100644 (file)
--- a/folly/Conv.cpp
+++ b/folly/Conv.cpp
@@ -69,7 +69,7 @@ static_assert(sizeof(unsigned long long) >= 8,
               "Wrong value for MaxString<unsigned long long>::value"
               ", please update.");
 
-#ifdef FOLLY_HAVE_INT128_T
+#if FOLLY_HAVE_INT128_T
 template <> const char *const MaxString<__uint128_t>::value =
   "340282366920938463463374607431768211455";
 #endif

diff --git a/folly/Conv.h b/folly/Conv.h
index d6553373fa0203dfb120466ce42a0b1a50a454c8..0af04cf6ca72ebf5af9af224d808f5bf1001ca06 100644 (file)
--- a/folly/Conv.h
+++ b/folly/Conv.h
@@ -702,7 +702,7 @@ struct HasLengthEstimator : std::false_type {};
 template <class Src>
 constexpr typename std::enable_if<
   !std::is_fundamental<Src>::value
-#ifdef FOLLY_HAVE_INT128_T
+#if FOLLY_HAVE_INT128_T
   // On OSX 10.10, is_fundamental<__int128> is false :-O
   && !std::is_same<__int128, Src>::value
   && !std::is_same<unsigned __int128, Src>::value