diff options
author | Erik de Castro Lopo <erik.decastrolopo@ambiata.com> | 2017-04-05 05:53:46 +1000 |
---|---|---|
committer | Ben Gamari <ben@smart-cactus.org> | 2017-04-28 22:35:04 -0400 |
commit | e5b3492f23c2296d0d8221e1787ee585331f726e (patch) | |
tree | e1d6797dadc265db7f97615ff71c65c3d83db9c0 /rts/posix/itimer/Pthread.c | |
parent | 6d14c1485cb570cbd183bcdc0f858d9a6dc1eb31 (diff) | |
download | haskell-e5b3492f23c2296d0d8221e1787ee585331f726e.tar.gz |
Enable new warning for fragile/incorrect CPP #if usage
The C code in the RTS now gets built with `-Wundef` and the Haskell code
(stages 1 and 2 only) with `-Wcpp-undef`. We now get warnings whereever
`#if` is used on undefined identifiers.
Test Plan: Validate on Linux and Windows
Reviewers: austin, angerman, simonmar, bgamari, Phyx
Reviewed By: bgamari
Subscribers: thomie, snowleopard
Differential Revision: https://phabricator.haskell.org/D3278
Diffstat (limited to 'rts/posix/itimer/Pthread.c')
-rw-r--r-- | rts/posix/itimer/Pthread.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/rts/posix/itimer/Pthread.c b/rts/posix/itimer/Pthread.c index 3b31fe4103..5c708ecf4d 100644 --- a/rts/posix/itimer/Pthread.c +++ b/rts/posix/itimer/Pthread.c @@ -65,7 +65,7 @@ #include <unistd.h> #include <fcntl.h> -#if HAVE_SYS_TIMERFD_H +#ifdef HAVE_SYS_TIMERFD_H #include <sys/timerfd.h> #define USE_TIMERFD_FOR_ITIMER 1 #else @@ -101,7 +101,7 @@ static void *itimer_thread_func(void *_handle_tick) uint64_t nticks; int timerfd = -1; -#if USE_TIMERFD_FOR_ITIMER +#if defined(USE_TIMERFD_FOR_ITIMER) && USE_TIMERFD_FOR_ITIMER struct itimerspec it; it.it_value.tv_sec = TimeToSeconds(itimer_interval); it.it_value.tv_nsec = TimeToNS(itimer_interval) % 1000000000; @@ -169,7 +169,7 @@ initTicker (Time interval, TickProc handle_tick) * to the thread we create so we can later join to it if requested */ if (! pthread_create(&thread, NULL, itimer_thread_func, (void*)handle_tick)) { -#if HAVE_PTHREAD_SETNAME_NP +#ifdef HAVE_PTHREAD_SETNAME_NP pthread_setname_np(thread, "ghc_ticker"); #endif } else { |