From: Jérémie Galarneau <jeremie.galarneau@efficios.com>
Date: Fri, 6 Sep 2019 17:33:18 +0000 (-0400)
Subject: Fix: time constants already defined on macOS
X-Git-Tag: v2.12.0-rc1~433
X-Git-Url: https://git.lttng.org/?p=lttng-tools.git;a=commitdiff_plain;h=efef32e9dabe51901ea2a0d1f19c69a9c37997eb;ds=sidebyside

Fix: time constants already defined on macOS

Include the compatibility time.h header in the common/time.h
to override any platform-defined constants. It seems that the
NSEC_PER_SEC (and other similar definitions) are defined to
nothing on macOS. This was already fixed in the compatibility
header in the past.

This also adds USEC_PER_SEC to the list of undef-initions
on macOS.

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
---

diff --git a/src/common/compat/time.h b/src/common/compat/time.h
index 49f503908..593a92ab0 100644
--- a/src/common/compat/time.h
+++ b/src/common/compat/time.h
@@ -36,6 +36,7 @@ typedef uint64_t timer_t;
 #undef NSEC_PER_SEC
 #undef NSEC_PER_MSEC
 #undef NSEC_PER_USEC
+#undef USEC_PER_SEC
 
 #endif /* __APPLE__ */
 
diff --git a/src/common/time.h b/src/common/time.h
index 60d0a9bd5..4cfdf856c 100644
--- a/src/common/time.h
+++ b/src/common/time.h
@@ -22,14 +22,13 @@
 #include <time.h>
 #include <stdbool.h>
 #include <common/macros.h>
+#include <common/compat/time.h>
 
 #define MSEC_PER_SEC    1000ULL
 #define NSEC_PER_SEC    1000000000ULL
 #define NSEC_PER_MSEC   1000000ULL
 #define NSEC_PER_USEC   1000ULL
-#ifndef USEC_PER_SEC
 #define USEC_PER_SEC    1000000ULL
-#endif
 #define USEC_PER_MSEC   1000ULL
 
 #define SEC_PER_MINUTE  60ULL