ext/date: Change return type of check_id_allowed from int to bool

Girgias · Girgias · commit 8a3dbb0401b2 · 2025-03-28T16:55:37.000Z
diff --git a/ext/date/php_date.c b/ext/date/php_date.c
@@ -5291,7 +5291,7 @@ PHP_METHOD(DatePeriod, getIterator)
 	zend_create_internal_iterator_zval(return_value, ZEND_THIS);
 }
 
-static int check_id_allowed(const char *id, zend_long what) /* {{{ */
+static bool check_id_allowed(const char *id, zend_long what) /* {{{ */
 {
 	if ((what & PHP_DATE_TIMEZONE_GROUP_AFRICA)     && strncasecmp(id, "Africa/",      7) == 0) return 1;
 	if ((what & PHP_DATE_TIMEZONE_GROUP_AMERICA)    && strncasecmp(id, "America/",     8) == 0) return 1;

Original file line number	Diff line number	Diff line change
`@@ -5291,7 +5291,7 @@ PHP_METHOD(DatePeriod, getIterator)`
`5291`	`5291`	`zend_create_internal_iterator_zval(return_value, ZEND_THIS);`
`5292`	`5292`	`}`
`5293`	`5293`
`5294`		`-static int check_id_allowed(const char id, zend_long what) / {{{ */`
	`5294`	`+static bool check_id_allowed(const char id, zend_long what) / {{{ */`
`5295`	`5295`	`{`
`5296`	`5296`	`if ((what & PHP_DATE_TIMEZONE_GROUP_AFRICA) && strncasecmp(id, "Africa/", 7) == 0) return 1;`
`5297`	`5297`	`if ((what & PHP_DATE_TIMEZONE_GROUP_AMERICA) && strncasecmp(id, "America/", 8) == 0) return 1;`