Skip to content

Commit 69b7d01

Browse files
Girgiasdstogov
authored andcommitted
Explicitly declare the char as signed in zend_ffi_val.
This causes issues down the line as char are unsigned on some platforms, e.g. ARM and cause a [-Wtype-limits] warning to be emitted.
1 parent 556573b commit 69b7d01

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

ext/ffi/php_ffi.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -198,7 +198,7 @@ typedef struct _zend_ffi_val {
198198
uint64_t u64;
199199
int64_t i64;
200200
zend_ffi_double d;
201-
char ch;
201+
signed char ch;
202202
struct {
203203
const char *str;
204204
size_t len;

0 commit comments

Comments
 (0)