From 77118fc925b3e84be02a80d8da6bbbb47f8c37e1 Mon Sep 17 00:00:00 2001 From: Peter Kokot Date: Wed, 5 Sep 2018 00:09:56 +0200 Subject: Remove HAVE_ASSERT_H The `` header file is part of the standard C89 headers [1] and on older systems there needed to be also a manual check if header is present. Since PHP requires at least C89 manual check and the `HAVE_ASSERT_H` symbol defined by Autoconf in configure.ac can be both removed [2]. This patch also removes unused includes where c files don't use the `assert()` macro. Refs: [1] https://port70.net/~nsz/c/c89/c89-draft.html#4.2 [2] https://git.savannah.gnu.org/cgit/autoconf.git/tree/lib/autoconf/headers.m4 --- main/php.h | 4 ---- 1 file changed, 4 deletions(-) (limited to 'main/php.h') diff --git a/main/php.h b/main/php.h index 697d3ff935..4134f7b3d4 100644 --- a/main/php.h +++ b/main/php.h @@ -123,7 +123,6 @@ typedef int pid_t; # endif #endif -#if HAVE_ASSERT_H #if PHP_DEBUG #undef NDEBUG #else @@ -132,9 +131,6 @@ typedef int pid_t; #endif #endif #include -#else /* HAVE_ASSERT_H */ -#define assert(expr) ((void) (0)) -#endif /* HAVE_ASSERT_H */ #define APACHE 0 -- cgit v1.2.1