mirrors/php-src
0
0
Fork 0
mirror of https://github.com/php/php-src.git synced 2024-11-24 10:24:11 +08:00
Code Issues Packages Projects Releases Wiki Activity

Remove HAVE_TIME_H

Browse Source 
The `<time.h>` header file is part of the standard C89 headers [1] and
on current systems can be included unconditionally.

Since PHP requires at least C89 or greater, the `HAVE_TIME_H` symbol
defined by Autoconf in ext/pdo_sqlite/config.m4 [2] can be ommitted and
simplifed.

Additionally, since PHP didn't define `HAVE_TIME_H` prior in the
configure.ac the occurrence of this symbol in cli can be removed.

Refs:
[1] https://port70.net/~nsz/c/c89/c89-draft.html#4.1.2
[2] https://git.savannah.gnu.org/cgit/autoconf.git/tree/lib/autoconf/headers.m4
This commit is contained in:
Peter Kokot 2018-09-18 19:26:19 +02:00
parent c1b649d448
commit a1b5698d4e
2 changed files with 0 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
ext/pdo_sqlite/config.m4
View File

@ -102,7 +102,6 @@ if test "$PHP_PDO_SQLITE" != "no"; then
PHP_ADD_INCLUDE($abs_srcdir/ext/sqlite3/libsqlite)
AC_CHECK_FUNCS(usleep nanosleep)
AC_CHECK_HEADERS(time.h)
fi
dnl Solaris fix

4
sapi/cli/php_cli_server.c
View File

@ -37,10 +37,6 @@
#include <unixlib/local.h>
#endif
#if HAVE_TIME_H
#include <time.h>
#endif
#if HAVE_SYS_TIME_H
#include <sys/time.h>
#endif