Skip to content

Commit f91f726

Browse files
Girgiascmb69
andcommitted
Drop unnecessary stdint and inttypes header checks
These are always available as of C99. Closes GH-5323 Co-authored-by: "Christoph M. Becker" <[email protected]>
1 parent 657f756 commit f91f726

File tree

19 files changed

+57
-709
lines changed

19 files changed

+57
-709
lines changed

Zend/zend_strtod_int.h

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -42,11 +42,7 @@
4242
https://wiki.php.net/rfc/remove_hex_support_in_numeric_strings */
4343
#define NO_HEX_FP 1
4444

45-
#if defined(HAVE_INTTYPES_H)
4645
#include <inttypes.h>
47-
#elif defined(HAVE_STDINT_H)
48-
#include <stdint.h>
49-
#endif
5046

5147
#ifndef HAVE_INT32_T
5248
# if SIZEOF_INT == 4

build/php.m4

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1044,9 +1044,7 @@ AC_DEFUN([_PHP_CHECK_SIZEOF], [
10441044
AC_RUN_IFELSE([AC_LANG_SOURCE([[#include <stdio.h>
10451045
#include <stdlib.h>
10461046
#include <stddef.h>
1047-
#ifdef HAVE_INTTYPES_H
10481047
#include <inttypes.h>
1049-
#endif
10501048
#ifdef HAVE_UNISTD_H
10511049
#include <unistd.h>
10521050
#endif
@@ -2421,9 +2419,7 @@ AC_DEFUN([PHP_CHECK_STDINT_TYPES], [
24212419
AC_CHECK_SIZEOF([long long])
24222420
AC_CHECK_SIZEOF([size_t])
24232421
AC_CHECK_TYPES([int8, int16, int32, int64, int8_t, int16_t, int32_t, int64_t, uint8, uint16, uint32, uint64, uint8_t, uint16_t, uint32_t, uint64_t, u_int8_t, u_int16_t, u_int32_t, u_int64_t], [], [], [
2424-
#if HAVE_STDINT_H
2425-
# include <stdint.h>
2426-
#endif
2422+
#include <stdint.h>
24272423
#if HAVE_SYS_TYPES_H
24282424
# include <sys/types.h>
24292425
#endif

ext/fileinfo/libmagic.patch

Lines changed: 40 additions & 44 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
diff -u libmagic.orig/apprentice.c libmagic/apprentice.c
22
--- libmagic.orig/apprentice.c 2019-02-20 03:35:27.000000000 +0100
3-
+++ libmagic/apprentice.c 2019-12-19 20:37:54.476535900 +0100
3+
+++ libmagic/apprentice.c 2020-04-07 22:25:10.486120900 +0200
44
@@ -29,6 +29,8 @@
55
* apprentice - make one pass through /etc/magic, learning its secrets.
66
*/
@@ -974,7 +974,7 @@ diff -u libmagic.orig/apprentice.c libmagic/apprentice.c
974974
}
975975
diff -u libmagic.orig/ascmagic.c libmagic/ascmagic.c
976976
--- libmagic.orig/ascmagic.c 2019-05-07 04:27:11.000000000 +0200
977-
+++ libmagic/ascmagic.c 2019-12-19 20:37:54.628894400 +0100
977+
+++ libmagic/ascmagic.c 2020-04-07 22:25:10.501740300 +0200
978978
@@ -96,7 +96,7 @@
979979
rv = file_ascmagic_with_encoding(ms, &bb,
980980
ubuf, ulen, code, type, text);
@@ -1005,7 +1005,7 @@ diff -u libmagic.orig/ascmagic.c libmagic/ascmagic.c
10051005
}
10061006
diff -u libmagic.orig/buffer.c libmagic/buffer.c
10071007
--- libmagic.orig/buffer.c 2019-05-07 04:27:11.000000000 +0200
1008-
+++ libmagic/buffer.c 2019-12-19 20:37:54.639865800 +0100
1008+
+++ libmagic/buffer.c 2020-04-07 22:25:10.501740300 +0200
10091009
@@ -31,19 +31,23 @@
10101010
#endif /* lint */
10111011

@@ -1062,7 +1062,7 @@ diff -u libmagic.orig/buffer.c libmagic/buffer.c
10621062

10631063
diff -u libmagic.orig/cdf.c libmagic/cdf.c
10641064
--- libmagic.orig/cdf.c 2019-02-20 03:35:27.000000000 +0100
1065-
+++ libmagic/cdf.c 2019-12-19 20:37:55.233790900 +0100
1065+
+++ libmagic/cdf.c 2020-04-07 22:25:10.517321000 +0200
10661066
@@ -43,7 +43,17 @@
10671067
#include <err.h>
10681068
#endif
@@ -1341,7 +1341,7 @@ diff -u libmagic.orig/cdf.c libmagic/cdf.c
13411341
#endif
13421342
diff -u libmagic.orig/cdf.h libmagic/cdf.h
13431343
--- libmagic.orig/cdf.h 2019-02-20 02:24:19.000000000 +0100
1344-
+++ libmagic/cdf.h 2019-12-19 20:37:55.431591900 +0100
1344+
+++ libmagic/cdf.h 2020-04-07 22:25:10.517321000 +0200
13451345
@@ -35,10 +35,10 @@
13461346
#ifndef _H_CDF_
13471347
#define _H_CDF_
@@ -1366,7 +1366,7 @@ diff -u libmagic.orig/cdf.h libmagic/cdf.h
13661366
#define CDF_SECID_FREE -1
13671367
diff -u libmagic.orig/cdf_time.c libmagic/cdf_time.c
13681368
--- libmagic.orig/cdf_time.c 2019-03-12 21:43:05.000000000 +0100
1369-
+++ libmagic/cdf_time.c 2019-12-19 20:37:55.483459500 +0100
1369+
+++ libmagic/cdf_time.c 2020-04-07 22:25:10.517321000 +0200
13701370
@@ -23,6 +23,7 @@
13711371
* ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
13721372
* POSSIBILITY OF SUCH DAMAGE.
@@ -1395,7 +1395,7 @@ diff -u libmagic.orig/cdf_time.c libmagic/cdf_time.c
13951395
(void)snprintf(buf, 26, "*Bad* %#16.16" INT64_T_FORMAT "x\n",
13961396
diff -u libmagic.orig/compress.c libmagic/compress.c
13971397
--- libmagic.orig/compress.c 2019-05-07 04:27:11.000000000 +0200
1398-
+++ libmagic/compress.c 2019-12-19 20:37:55.501422500 +0100
1398+
+++ libmagic/compress.c 2020-04-07 22:25:10.517321000 +0200
13991399
@@ -45,13 +45,11 @@
14001400
#endif
14011401
#include <string.h>
@@ -1545,7 +1545,7 @@ diff -u libmagic.orig/compress.c libmagic/compress.c
15451545
+#endif
15461546
diff -u libmagic.orig/der.c libmagic/der.c
15471547
--- libmagic.orig/der.c 2019-02-20 03:35:27.000000000 +0100
1548-
+++ libmagic/der.c 2019-12-19 20:37:55.522331900 +0100
1548+
+++ libmagic/der.c 2020-04-07 22:25:10.517321000 +0200
15491549
@@ -51,7 +51,9 @@
15501550
#include "magic.h"
15511551
#include "der.h"
@@ -1575,7 +1575,7 @@ diff -u libmagic.orig/der.c libmagic/der.c
15751575
snprintf(buf + z, blen - z, "%.2x", d[i]);
15761576
diff -u libmagic.orig/elfclass.h libmagic/elfclass.h
15771577
--- libmagic.orig/elfclass.h 2019-02-20 02:30:19.000000000 +0100
1578-
+++ libmagic/elfclass.h 2019-12-19 20:37:55.539285200 +0100
1578+
+++ libmagic/elfclass.h 2020-04-07 22:25:10.517321000 +0200
15791579
@@ -41,7 +41,7 @@
15801580
return toomany(ms, "program headers", phnum);
15811581
flags |= FLAGS_IS_CORE;
@@ -1605,7 +1605,7 @@ diff -u libmagic.orig/elfclass.h libmagic/elfclass.h
16051605
CAST(int, elf_getu16(swap, elfhdr.e_shstrndx)),
16061606
diff -u libmagic.orig/encoding.c libmagic/encoding.c
16071607
--- libmagic.orig/encoding.c 2019-04-15 18:48:41.000000000 +0200
1608-
+++ libmagic/encoding.c 2019-12-19 20:37:55.547264800 +0100
1608+
+++ libmagic/encoding.c 2020-04-07 22:25:10.517321000 +0200
16091609
@@ -89,13 +89,13 @@
16101610
*code_mime = "binary";
16111611

@@ -1636,7 +1636,7 @@ diff -u libmagic.orig/encoding.c libmagic/encoding.c
16361636
}
16371637
diff -u libmagic.orig/file.h libmagic/file.h
16381638
--- libmagic.orig/file.h 2019-05-07 04:27:11.000000000 +0200
1639-
+++ libmagic/file.h 2020-03-29 18:05:08.870164300 +0200
1639+
+++ libmagic/file.h 2020-04-22 20:15:46.790840100 +0200
16401640
@@ -33,18 +33,9 @@
16411641
#ifndef __file_h__
16421642
#define __file_h__
@@ -1658,7 +1658,7 @@ diff -u libmagic.orig/file.h libmagic/file.h
16581658
#ifdef _WIN64
16591659
#define SIZE_T_FORMAT "I64"
16601660
#else
1661-
@@ -57,19 +48,34 @@
1661+
@@ -57,19 +48,28 @@
16621662
#define INT64_T_FORMAT "ll"
16631663
#define INTMAX_T_FORMAT "j"
16641664
#endif
@@ -1668,23 +1668,19 @@ diff -u libmagic.orig/file.h libmagic/file.h
16681668
#include <stdio.h> /* Include that here, to make sure __P gets defined */
16691669
#include <errno.h>
16701670
#include <fcntl.h> /* For open and flags */
1671-
+#ifdef HAVE_STDINT_H
1671+
-#ifdef HAVE_INTTYPES_H
1672+
-#include <inttypes.h>
1673+
+
16721674
+#ifndef __STDC_LIMIT_MACROS
1673-
+#define __STDC_LIMIT_MACROS
1674-
+#endif
1675-
+#ifndef __STDC_FORMAT_MACROS
1676-
+#define __STDC_FORMAT_MACROS
1677-
+#endif
1678-
+#include <stdint.h>
1679-
+#endif
1680-
#ifdef HAVE_INTTYPES_H
1681-
#include <inttypes.h>
1675+
+# define __STDC_LIMIT_MACROS
16821676
#endif
16831677
-#include <regex.h>
16841678
-#include <time.h>
1685-
+#ifdef PHP_WIN32
1686-
+#include "win32/php_stdint.h"
1679+
+#ifndef __STDC_FORMAT_MACROS
1680+
+# define __STDC_FORMAT_MACROS
16871681
+#endif
1682+
+#include <stdint.h>
1683+
+#include <inttypes.h>
16881684
+
16891685
+#include "php.h"
16901686
+#include "ext/standard/php_string.h"
@@ -1698,7 +1694,7 @@ diff -u libmagic.orig/file.h libmagic/file.h
16981694
#include <sys/param.h>
16991695
#endif
17001696
/* Do this here and now, because struct stat gets re-defined on solaris */
1701-
@@ -82,7 +88,7 @@
1697+
@@ -82,7 +82,7 @@
17021698
#define MAGIC "/etc/magic"
17031699
#endif
17041700

@@ -1707,7 +1703,7 @@ diff -u libmagic.orig/file.h libmagic/file.h
17071703
#define PATHSEP ';'
17081704
#else
17091705
#define PATHSEP ':'
1710-
@@ -116,12 +122,6 @@
1706+
@@ -116,12 +116,6 @@
17111707
#endif
17121708
#endif
17131709

@@ -1720,7 +1716,7 @@ diff -u libmagic.orig/file.h libmagic/file.h
17201716
#ifndef MIN
17211717
#define MIN(a,b) (((a) < (b)) ? (a) : (b))
17221718
#endif
1723-
@@ -150,10 +150,10 @@
1719+
@@ -150,10 +144,10 @@
17241720

17251721
struct buffer {
17261722
int fd;
@@ -1733,7 +1729,7 @@ diff -u libmagic.orig/file.h libmagic/file.h
17331729
void *ebuf;
17341730
size_t elen;
17351731
};
1736-
@@ -243,7 +243,7 @@
1732+
@@ -243,7 +237,7 @@
17371733
#define FILE_DER 48
17381734
#define FILE_NAMES_SIZE 49 /* size of array to contain all names */
17391735

@@ -1742,7 +1738,7 @@ diff -u libmagic.orig/file.h libmagic/file.h
17421738
((t) == FILE_STRING || \
17431739
(t) == FILE_PSTRING || \
17441740
(t) == FILE_BESTRING16 || \
1745-
@@ -447,28 +447,23 @@
1741+
@@ -447,28 +441,23 @@
17461742
/* Type for Unicode characters */
17471743
typedef unsigned long unichar;
17481744

@@ -1776,7 +1772,7 @@ diff -u libmagic.orig/file.h libmagic/file.h
17761772
protected int file_zmagic(struct magic_set *, const struct buffer *,
17771773
const char *);
17781774
#endif
1779-
@@ -491,13 +486,9 @@
1775+
@@ -491,13 +480,9 @@
17801776
protected void file_badread(struct magic_set *);
17811777
protected void file_badseek(struct magic_set *);
17821778
protected void file_oomem(struct magic_set *, size_t);
@@ -1793,7 +1789,7 @@ diff -u libmagic.orig/file.h libmagic/file.h
17931789
protected void file_showstr(FILE *, const char *, size_t);
17941790
protected size_t file_mbswidth(const char *);
17951791
protected const char *file_getbuffer(struct magic_set *);
1796-
@@ -513,34 +504,13 @@
1792+
@@ -513,34 +498,13 @@
17971793
size_t);
17981794
#endif /* __EMX__ */
17991795

@@ -1831,15 +1827,15 @@ diff -u libmagic.orig/file.h libmagic/file.h
18311827

18321828
typedef struct {
18331829
char *buf;
1834-
@@ -550,28 +520,13 @@
1830+
@@ -550,28 +514,13 @@
18351831
protected file_pushbuf_t *file_push_buffer(struct magic_set *);
18361832
protected char *file_pop_buffer(struct magic_set *, file_pushbuf_t *);
18371833

18381834
-#ifndef COMPILE_ONLY
18391835
extern const char *file_names[];
18401836
extern const size_t file_nnames;
18411837
-#endif
1842-
1838+
-
18431839
-#ifndef HAVE_PREAD
18441840
-ssize_t pread(int, void *, size_t, off_t);
18451841
-#endif
@@ -1852,7 +1848,7 @@ diff -u libmagic.orig/file.h libmagic/file.h
18521848
-#ifndef HAVE_DPRINTF
18531849
-int dprintf(int, const char *, ...);
18541850
-#endif
1855-
-
1851+
18561852
-#ifndef HAVE_STRLCPY
18571853
+#ifndef strlcpy
18581854
size_t strlcpy(char *, const char *, size_t);
@@ -1862,7 +1858,7 @@ diff -u libmagic.orig/file.h libmagic/file.h
18621858
size_t strlcat(char *, const char *, size_t);
18631859
#endif
18641860
#ifndef HAVE_STRCASESTR
1865-
@@ -587,39 +542,6 @@
1861+
@@ -587,39 +536,6 @@
18661862
#ifndef HAVE_ASCTIME_R
18671863
char *asctime_r(const struct tm *, char *);
18681864
#endif
@@ -1902,7 +1898,7 @@ diff -u libmagic.orig/file.h libmagic/file.h
19021898

19031899
#if defined(HAVE_MMAP) && defined(HAVE_SYS_MMAN_H) && !defined(QUICK)
19041900
#define QUICK
1905-
@@ -645,6 +567,18 @@
1901+
@@ -645,6 +561,18 @@
19061902
#else
19071903
#define FILE_RCSID(id)
19081904
#endif
@@ -1923,7 +1919,7 @@ diff -u libmagic.orig/file.h libmagic/file.h
19231919
#endif
19241920
diff -u libmagic.orig/fsmagic.c libmagic/fsmagic.c
19251921
--- libmagic.orig/fsmagic.c 2019-05-07 04:26:48.000000000 +0200
1926-
+++ libmagic/fsmagic.c 2019-12-19 20:37:55.740986600 +0100
1922+
+++ libmagic/fsmagic.c 2020-04-07 22:25:10.532971400 +0200
19271923
@@ -66,26 +66,10 @@
19281924
# define minor(dev) ((dev) & 0xff)
19291925
#endif
@@ -2216,7 +2212,7 @@ diff -u libmagic.orig/fsmagic.c libmagic/fsmagic.c
22162212
case S_IFSOCK:
22172213
diff -u libmagic.orig/funcs.c libmagic/funcs.c
22182214
--- libmagic.orig/funcs.c 2019-05-07 04:27:11.000000000 +0200
2219-
+++ libmagic/funcs.c 2020-03-30 15:56:57.404908500 +0200
2215+
+++ libmagic/funcs.c 2020-04-14 17:15:50.737587100 +0200
22202216
@@ -31,87 +31,80 @@
22212217
#endif /* lint */
22222218

@@ -2588,7 +2584,7 @@ diff -u libmagic.orig/funcs.c libmagic/funcs.c
25882584

25892585
diff -u libmagic.orig/magic.c libmagic/magic.c
25902586
--- libmagic.orig/magic.c 2019-05-07 04:27:11.000000000 +0200
2591-
+++ libmagic/magic.c 2019-12-19 20:37:55.783491600 +0100
2587+
+++ libmagic/magic.c 2020-04-07 22:25:10.532971400 +0200
25922588
@@ -25,11 +25,6 @@
25932589
* SUCH DAMAGE.
25942590
*/
@@ -3052,8 +3048,8 @@ diff -u libmagic.orig/magic.c libmagic/magic.c
30523048
public const char *
30533049
magic_error(struct magic_set *ms)
30543050
diff -u libmagic.orig/magic.h libmagic/magic.h
3055-
--- libmagic.orig/magic.h 2020-03-30 16:09:56.288564100 +0200
3056-
+++ libmagic/magic.h 2019-12-19 20:37:55.792489700 +0100
3051+
--- libmagic.orig/magic.h 2020-04-22 20:17:15.432186600 +0200
3052+
+++ libmagic/magic.h 2020-04-07 22:25:10.548560600 +0200
30573053
@@ -124,6 +124,7 @@
30583054

30593055
const char *magic_getpath(const char *, int);
@@ -3064,7 +3060,7 @@ diff -u libmagic.orig/magic.h libmagic/magic.h
30643060

30653061
diff -u libmagic.orig/print.c libmagic/print.c
30663062
--- libmagic.orig/print.c 2019-03-12 21:43:05.000000000 +0100
3067-
+++ libmagic/print.c 2019-12-19 20:37:55.808452700 +0100
3063+
+++ libmagic/print.c 2020-04-07 22:25:10.548560600 +0200
30683064
@@ -28,6 +28,7 @@
30693065
/*
30703066
* print.c - debugging printout routines
@@ -3138,7 +3134,7 @@ diff -u libmagic.orig/print.c libmagic/print.c
31383134
goto out;
31393135
diff -u libmagic.orig/readcdf.c libmagic/readcdf.c
31403136
--- libmagic.orig/readcdf.c 2019-03-12 21:43:05.000000000 +0100
3141-
+++ libmagic/readcdf.c 2020-03-21 14:20:14.170673900 +0100
3137+
+++ libmagic/readcdf.c 2020-04-07 22:25:10.548560600 +0200
31423138
@@ -31,7 +31,11 @@
31433139

31443140
#include <assert.h>
@@ -3257,7 +3253,7 @@ diff -u libmagic.orig/readcdf.c libmagic/readcdf.c
32573253
if (i != -1)
32583254
diff -u libmagic.orig/softmagic.c libmagic/softmagic.c
32593255
--- libmagic.orig/softmagic.c 2019-05-17 04:24:59.000000000 +0200
3260-
+++ libmagic/softmagic.c 2020-03-25 17:15:23.794665300 +0100
3256+
+++ libmagic/softmagic.c 2020-04-07 22:25:10.548560600 +0200
32613257
@@ -43,6 +43,10 @@
32623258
#include <time.h>
32633259
#include "der.h"

ext/fileinfo/libmagic/file.h

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -52,21 +52,15 @@
5252
#include <stdio.h> /* Include that here, to make sure __P gets defined */
5353
#include <errno.h>
5454
#include <fcntl.h> /* For open and flags */
55-
#ifdef HAVE_STDINT_H
55+
5656
#ifndef __STDC_LIMIT_MACROS
57-
#define __STDC_LIMIT_MACROS
57+
# define __STDC_LIMIT_MACROS
5858
#endif
5959
#ifndef __STDC_FORMAT_MACROS
60-
#define __STDC_FORMAT_MACROS
60+
# define __STDC_FORMAT_MACROS
6161
#endif
6262
#include <stdint.h>
63-
#endif
64-
#ifdef HAVE_INTTYPES_H
6563
#include <inttypes.h>
66-
#endif
67-
#ifdef PHP_WIN32
68-
#include "win32/php_stdint.h"
69-
#endif
7064

7165
#include "php.h"
7266
#include "ext/standard/php_string.h"

ext/mysqlnd/config-win.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ This file is public domain and comes with NO WARRANTY of any kind */
1313
#include <io.h>
1414
#include <malloc.h>
1515

16-
#include <win32/php_stdint.h>
16+
#include <stdint.h>
1717

1818
#ifndef HAVE_INT8_T
1919
#define HAVE_INT8_T

ext/sockets/conversions.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
# include <sys/ioctl.h>
1818
# include <net/if.h>
1919
#else
20-
# include <win32/php_stdint.h>
20+
# include <stdint.h>
2121
#endif
2222

2323
#include <limits.h>

ext/standard/image.c

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,9 +26,6 @@
2626
#include <unistd.h>
2727
#endif
2828
#include "php_image.h"
29-
#ifdef PHP_WIN32
30-
#include "win32/php_stdint.h"
31-
#endif
3229

3330
#if HAVE_ZLIB && !defined(COMPILE_DL_ZLIB)
3431
#include "zlib.h"

0 commit comments

Comments
 (0)