configure: add '-Wundef' to warning flags

Change-Id: If47c16121b1679862e7a5f75fce70c7d9973e92e
Signed-off-by: Michael Jeanson <mjeanson@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

diff --git a/configure.ac b/configure.ac
index c04a3cbb88fe27f3bd8bf86d69a7b2062abcef8f..5a63255c4549435a3594c08d996642ba73b130ea 100644 (file)
--- a/configure.ac
+++ b/configure.ac
@@ -60,6 +60,7 @@ AC_TYPE_UINT8_T
 m4_define([WARN_FLAGS_COMMON_LIST], [ dnl
   -Wall dnl
   -Wnull-dereference dnl
+  -Wundef dnl
   dnl We currently get this warning when building with Clang:
   dnl
   dnl /usr/include/setjmp.h:54:12: error: declaration of built-in function '__sigsetjmp' requires the declaration of the 'jmp_buf' type, commonly provided in the header <setjmp.h>. [-Werror,-Wincomplete-setjmp-declaration]

diff --git a/src/bin/lttng-sessiond/modprobe.cpp b/src/bin/lttng-sessiond/modprobe.cpp
index 4c1d62b486e2b37f4181f398a7c03da58289b65d..9f6c4a9c3edabb8e64b39de0c8087d54bee0d669 100644 (file)
--- a/src/bin/lttng-sessiond/modprobe.cpp
+++ b/src/bin/lttng-sessiond/modprobe.cpp
@@ -242,7 +242,7 @@ static struct kern_modules_param *probes;
 static int nr_probes;
 static int probes_capacity;
 
-#if HAVE_KMOD
+#ifdef HAVE_KMOD
 #include <libkmod.h>
 
 /**

diff --git a/src/bin/lttng-sessiond/ust-consumer.h b/src/bin/lttng-sessiond/ust-consumer.h
index 5862c4837cb4b77ae391f4f96839cbe60103d656..4f9fca386825641ec4396c2b4aa84e87c98fa19d 100644 (file)
--- a/src/bin/lttng-sessiond/ust-consumer.h
+++ b/src/bin/lttng-sessiond/ust-consumer.h
@@ -32,7 +32,7 @@ int ust_consumer_send_stream_to_ust(struct ust_app *app,
 int ust_consumer_send_channel_to_ust(struct ust_app *app,
                struct ust_app_session *ua_sess, struct ust_app_channel *channel);
 
-#if HAVE_LIBLTTNG_UST_CTL
+#ifdef HAVE_LIBLTTNG_UST_CTL
 int ust_consumer_metadata_request(struct consumer_socket *sock);
 #else
 static inline

diff --git a/src/bin/lttng/commands/add_context.cpp b/src/bin/lttng/commands/add_context.cpp
index 6ea53709c01df87f7d4dc9c0345f1396e9e8d810..4e4bb9fac110d5823247f56983455fad38bcf523 100644 (file)
--- a/src/bin/lttng/commands/add_context.cpp
+++ b/src/bin/lttng/commands/add_context.cpp
@@ -288,7 +288,7 @@ const struct ctx_opts {
        { (char *) "need_reschedule", CONTEXT_NEED_RESCHEDULE },
        { (char *) "migratable", CONTEXT_MIGRATABLE },
        { (char *) "callstack-kernel", CONTEXT_CALLSTACK_KERNEL },
-#if HAVE_MODULES_USERSPACE_CALLSTACK_CONTEXT
+#ifdef HAVE_MODULES_USERSPACE_CALLSTACK_CONTEXT
        { (char *) "callstack-user", CONTEXT_CALLSTACK_USER },
 #endif
        { (char *) "cgroup_ns", CONTEXT_CGROUP_NS },

diff --git a/src/common/compat/poll.cpp b/src/common/compat/poll.cpp
index 0c40c4511d266731d7a556daab9aebb172de9982..910b0f62aee469dfc1691e3b7eb212c01d1fdeb7 100644 (file)
--- a/src/common/compat/poll.cpp
+++ b/src/common/compat/poll.cpp
@@ -17,7 +17,7 @@
 
 #include "poll.h"
 
-#if HAVE_EPOLL
+#ifdef HAVE_EPOLL
 
 #include <fcntl.h>
 #include <limits.h>

diff --git a/src/common/compat/string.h b/src/common/compat/string.h
index fd4f2f021c38c29e7df97567270e3b103cbff0b6..4a96cb1456ffd2a928d18439007cfc3959b2b5e5 100644 (file)
--- a/src/common/compat/string.h
+++ b/src/common/compat/string.h
@@ -109,7 +109,7 @@ static inline int lttng_fls(int val)
 }
 #endif /* HAVE_FLS */
 
-#if HAVE_MEMRCHR
+#ifdef HAVE_MEMRCHR
 static inline
 void *lttng_memrchr(const void *s, int c, size_t n)
 {

diff --git a/src/common/fd-tracker/utils-poll.cpp b/src/common/fd-tracker/utils-poll.cpp
index 4949d8160f104bad611ed35f928a76fbb9141194..6a0f2d9bd28c7c6178af1af8d43889b9bbb689e7 100644 (file)
--- a/src/common/fd-tracker/utils-poll.cpp
+++ b/src/common/fd-tracker/utils-poll.cpp
@@ -9,7 +9,7 @@
 
 #include "utils.h"
 
-#if HAVE_EPOLL
+#ifdef HAVE_EPOLL
 
 struct create_args {
        struct lttng_poll_event *events;

diff --git a/tests/regression/ust/linking/demo.c b/tests/regression/ust/linking/demo.c
index ca2c759caaa3b65271ecd489dbb18e045eda4704..74d1d22a601f11ab7840b59e41a54fa69e82fb52 100644 (file)
--- a/tests/regression/ust/linking/demo.c
+++ b/tests/regression/ust/linking/demo.c
@@ -20,7 +20,7 @@
 
 #define TRACEPOINT_DEFINE
 
-#if TEST_DYNAMIC_LINKAGE
+#ifdef TEST_DYNAMIC_LINKAGE
 #define TRACEPOINT_PROBE_DYNAMIC_LINKAGE
 #endif