From c2fe4b0a6865ea1b9cb25c89475307d9b9600928 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C5=82=20Janiszewski?=
 <janisozaur@users.noreply.github.com>
Date: Sat, 19 Jun 2021 18:53:36 +0200
Subject: [PATCH] fix: Address -Wundef warning for SENTRY_UNITTEST defines
 (#549)

---
 src/modulefinder/sentry_modulefinder_linux.h | 2 +-
 src/sentry_core.c                            | 2 +-
 src/sentry_envelope.c                        | 2 +-
 src/sentry_envelope.h                        | 2 +-
 src/sentry_ratelimiter.c                     | 2 +-
 src/sentry_ratelimiter.h                     | 2 +-
 src/sentry_unix_pageallocator.c              | 2 +-
 src/sentry_unix_pageallocator.h              | 2 +-
 8 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/src/modulefinder/sentry_modulefinder_linux.h b/src/modulefinder/sentry_modulefinder_linux.h
index 71654521c..5204158c0 100644
--- a/src/modulefinder/sentry_modulefinder_linux.h
+++ b/src/modulefinder/sentry_modulefinder_linux.h
@@ -29,7 +29,7 @@ typedef struct {
     uint8_t num_mappings;
 } sentry_module_t;
 
-#if SENTRY_UNITTEST
+#ifdef SENTRY_UNITTEST
 bool sentry__procmaps_read_ids_from_elf(
     sentry_value_t value, const sentry_module_t *module);
 
diff --git a/src/sentry_core.c b/src/sentry_core.c
index 704d8549c..3774b99d8 100644
--- a/src/sentry_core.c
+++ b/src/sentry_core.c
@@ -434,7 +434,7 @@ sentry_handle_exception(const sentry_ucontext_t *uctx)
 sentry_uuid_t
 sentry__new_event_id(void)
 {
-#if SENTRY_UNITTEST
+#ifdef SENTRY_UNITTEST
     return sentry_uuid_from_string("4c035723-8638-4c3a-923f-2ab9d08b4018");
 #else
     return sentry_uuid_new_v4();
diff --git a/src/sentry_envelope.c b/src/sentry_envelope.c
index 7bd621c0d..71cef6ed6 100644
--- a/src/sentry_envelope.c
+++ b/src/sentry_envelope.c
@@ -410,7 +410,7 @@ sentry_envelope_write_to_file(
     return rv;
 }
 
-#if SENTRY_UNITTEST
+#ifdef SENTRY_UNITTEST
 size_t
 sentry__envelope_get_item_count(const sentry_envelope_t *envelope)
 {
diff --git a/src/sentry_envelope.h b/src/sentry_envelope.h
index efd31a771..ac5c69136 100644
--- a/src/sentry_envelope.h
+++ b/src/sentry_envelope.h
@@ -86,7 +86,7 @@ MUST_USE int sentry_envelope_write_to_path(
     const sentry_envelope_t *envelope, const sentry_path_t *path);
 
 // these for now are only needed for tests
-#if SENTRY_UNITTEST
+#ifdef SENTRY_UNITTEST
 size_t sentry__envelope_get_item_count(const sentry_envelope_t *envelope);
 const sentry_envelope_item_t *sentry__envelope_get_item(
     const sentry_envelope_t *envelope, size_t idx);
diff --git a/src/sentry_ratelimiter.c b/src/sentry_ratelimiter.c
index 025f69707..e1dd8c855 100644
--- a/src/sentry_ratelimiter.c
+++ b/src/sentry_ratelimiter.c
@@ -99,7 +99,7 @@ sentry__rate_limiter_free(sentry_rate_limiter_t *rl)
     sentry_free(rl);
 }
 
-#if SENTRY_UNITTEST
+#ifdef SENTRY_UNITTEST
 uint64_t
 sentry__rate_limiter_get_disabled_until(
     const sentry_rate_limiter_t *rl, int category)
diff --git a/src/sentry_ratelimiter.h b/src/sentry_ratelimiter.h
index 0a17754d3..58a666121 100644
--- a/src/sentry_ratelimiter.h
+++ b/src/sentry_ratelimiter.h
@@ -41,7 +41,7 @@ bool sentry__rate_limiter_update_from_http_retry_after(
 bool sentry__rate_limiter_is_disabled(
     const sentry_rate_limiter_t *rl, int category);
 
-#if SENTRY_UNITTEST
+#ifdef SENTRY_UNITTEST
 /**
  * The rate limiters state is completely opaque. Unless in tests, where we would
  * want to actually peek into the specific rate limiting `category`.
diff --git a/src/sentry_unix_pageallocator.c b/src/sentry_unix_pageallocator.c
index 09653a60d..0226c9b09 100644
--- a/src/sentry_unix_pageallocator.c
+++ b/src/sentry_unix_pageallocator.c
@@ -125,7 +125,7 @@ sentry__page_allocator_alloc(size_t size)
     return rv;
 }
 
-#if SENTRY_UNITTEST
+#ifdef SENTRY_UNITTEST
 void
 sentry__page_allocator_disable(void)
 {
diff --git a/src/sentry_unix_pageallocator.h b/src/sentry_unix_pageallocator.h
index 290192bc0..efc5b68b3 100644
--- a/src/sentry_unix_pageallocator.h
+++ b/src/sentry_unix_pageallocator.h
@@ -21,7 +21,7 @@ void sentry__page_allocator_enable(void);
  */
 void *sentry__page_allocator_alloc(size_t size);
 
-#if SENTRY_UNITTEST
+#ifdef SENTRY_UNITTEST
 /**
  * This disables the page allocator, which invalidates every allocation that was
  * done through it. Therefore it is only safe to use in unit tests