tree-wide: Replace zval_is_true() by zend_is_true() (#20065)

TimWolla · web-flow · commit 985d68150179 · 2025-10-07T20:44:32.000+02:00
* tree-wide: Replace `zval_is_true()` by `zend_is_true()`

The former is a direct alias of the latter which is much more often used.

* zend_operators: Remove `zval_is_true()`
diff --git a/UPGRADING.INTERNALS b/UPGRADING.INTERNALS
@@ -20,6 +20,8 @@ PHP 8.6 INTERNALS UPGRADE NOTES
   . The misnamed ZVAL_IS_NULL() has been removed. Use Z_ISNULL() instead.
   . New zend_class_entry.ce_flags2 and zend_function.fn_flags2 fields were
     added, given the primary flags were running out of bits.
+  . The zval_is_true() alias of zend_is_true() has been removed. Call
+    zend_is_true() directly instead.
 
 ========================
 2. Build system changes
diff --git a/Zend/zend_compile.c b/Zend/zend_compile.c
@@ -12062,7 +12062,7 @@ bool zend_try_ct_eval_cast(zval *result, uint32_t type, zval *op1)
 	}
 	switch (type) {
 		case _IS_BOOL:
-			ZVAL_BOOL(result, zval_is_true(op1));
+			ZVAL_BOOL(result, zend_is_true(op1));
 			return true;
 		case IS_LONG:
 			if (Z_TYPE_P(op1) == IS_DOUBLE && !ZEND_DOUBLE_FITS_LONG(Z_DVAL_P((op1)))) {
diff --git a/Zend/zend_operators.c b/Zend/zend_operators.c
@@ -1593,7 +1593,7 @@ ZEND_API zend_result ZEND_FASTCALL boolean_xor_function(zval *result, zval *op1,
 				}
 			}
 			ZEND_TRY_BINARY_OP1_OBJECT_OPERATION(ZEND_BOOL_XOR);
-			op1_val = zval_is_true(op1);
+			op1_val = zend_is_true(op1);
 		}
 	} while (0);
 	do {
@@ -1613,7 +1613,7 @@ ZEND_API zend_result ZEND_FASTCALL boolean_xor_function(zval *result, zval *op1,
 				}
 			}
 			ZEND_TRY_BINARY_OP2_OBJECT_OPERATION(ZEND_BOOL_XOR);
-			op2_val = zval_is_true(op2);
+			op2_val = zend_is_true(op2);
 		}
 	} while (0);
 
@@ -1641,7 +1641,7 @@ ZEND_API zend_result ZEND_FASTCALL boolean_not_function(zval *result, zval *op1)
 		}
 		ZEND_TRY_UNARY_OBJECT_OPERATION(ZEND_BOOL_NOT);
 
-		ZVAL_BOOL(result, !zval_is_true(op1));
+		ZVAL_BOOL(result, !zend_is_true(op1));
 	}
 	return SUCCESS;
 }
@@ -2433,13 +2433,13 @@ ZEND_API int ZEND_FASTCALL zend_compare(zval *op1, zval *op2) /* {{{ */
 							converted = true;
 						}
 					} else if (Z_TYPE_P(op1) < IS_TRUE) {
-						return zval_is_true(op2) ? -1 : 0;
+						return zend_is_true(op2) ? -1 : 0;
 					} else if (Z_TYPE_P(op1) == IS_TRUE) {
-						return zval_is_true(op2) ? 0 : 1;
+						return zend_is_true(op2) ? 0 : 1;
 					} else if (Z_TYPE_P(op2) < IS_TRUE) {
-						return zval_is_true(op1) ? 1 : 0;
+						return zend_is_true(op1) ? 1 : 0;
 					} else if (Z_TYPE_P(op2) == IS_TRUE) {
-						return zval_is_true(op1) ? 0 : -1;
+						return zend_is_true(op1) ? 0 : -1;
 					} else {
 						op1 = _zendi_convert_scalar_to_number_silent(op1, &op1_copy);
 						op2 = _zendi_convert_scalar_to_number_silent(op2, &op2_copy);
diff --git a/Zend/zend_operators.h b/Zend/zend_operators.h
@@ -404,9 +404,6 @@ static zend_always_inline bool try_convert_to_string(zval *op) {
 ZEND_API bool ZEND_FASTCALL zend_is_true(const zval *op);
 ZEND_API bool ZEND_FASTCALL zend_object_is_true(const zval *op);
 
-#define zval_is_true(op) \
-	zend_is_true(op)
-
 static zend_always_inline bool i_zend_is_true(const zval *op)
 {
 	bool result = 0;
diff --git a/ext/mysqli/mysqli.c b/ext/mysqli/mysqli.c
@@ -316,7 +316,7 @@ static int mysqli_object_has_property(zend_object *object, zend_string *name, in
 				zval rv;
 				zval *value = mysqli_read_property(object, name, BP_VAR_IS, cache_slot, &rv);
 				if (value != &EG(uninitialized_zval)) {
-					has_property = zval_is_true(value);
+					has_property = zend_is_true(value);
 					zval_ptr_dtor(value);
 				}
 				break;
diff --git a/ext/pdo/pdo_dbh.c b/ext/pdo/pdo_dbh.c
@@ -805,7 +805,7 @@ PDO_API bool pdo_get_bool_param(bool *bval, const zval *value)
 			*bval = false;
 			return true;
 		case IS_LONG:
-			*bval = zval_is_true(value);
+			*bval = zend_is_true(value);
 			return true;
 		case IS_STRING: /* TODO Should string be allowed? */
 		default:
diff --git a/ext/pdo_odbc/odbc_stmt.c b/ext/pdo_odbc/odbc_stmt.c
@@ -803,7 +803,7 @@ static int odbc_stmt_set_param(pdo_stmt_t *stmt, zend_long attr, zval *val)
 			return 0;
 
 		case PDO_ODBC_ATTR_ASSUME_UTF8:
-			S->assume_utf8 = zval_is_true(val);
+			S->assume_utf8 = zend_is_true(val);
 			return 0;
 		default:
 			strcpy(S->einfo.last_err_msg, "Unknown Attribute");
diff --git a/ext/pgsql/pgsql.c b/ext/pgsql/pgsql.c
@@ -4967,7 +4967,7 @@ PHP_PGSQL_API zend_result php_pgsql_convert(PGconn *pg_link, const zend_string *
 			break; /* break out for() */
 		}
 
-		if (zval_is_true(is_enum)) {
+		if (zend_is_true(is_enum)) {
 			/* enums need to be treated like strings */
 			data_type = PG_TEXT;
 		} else {
diff --git a/ext/session/session.c b/ext/session/session.c
@@ -1858,15 +1858,15 @@ PHP_FUNCTION(session_set_cookie_params)
 					domain = zval_get_string(value);
 					found++;
 				} else if (zend_string_equals_literal_ci(key, "secure")) {
-					secure = zval_is_true(value);
+					secure = zend_is_true(value);
 					secure_null = 0;
 					found++;
 				} else if (zend_string_equals_literal_ci(key, "partitioned")) {
-					partitioned = zval_is_true(value);
+					partitioned = zend_is_true(value);
 					partitioned_null = 0;
 					found++;
 				} else if (zend_string_equals_literal_ci(key, "httponly")) {
-					httponly = zval_is_true(value);
+					httponly = zend_is_true(value);
 					httponly_null = 0;
 					found++;
 				} else if (zend_string_equals_literal_ci(key, "samesite")) {
diff --git a/ext/sockets/multicast.c b/ext/sockets/multicast.c
@@ -293,7 +293,7 @@ int php_do_setsockopt_ip_mcast(php_socket *php_sock,
 		goto dosockopt;
 
 	case IP_MULTICAST_LOOP:
-		ipv4_mcast_ttl_lback = (unsigned char) zval_is_true(arg4);
+		ipv4_mcast_ttl_lback = (unsigned char) zend_is_true(arg4);
 		goto ipv4_loop_ttl;
 
 	case IP_MULTICAST_TTL:
@@ -357,7 +357,7 @@ int php_do_setsockopt_ipv6_mcast(php_socket *php_sock,
 		goto dosockopt;
 
 	case IPV6_MULTICAST_LOOP:
-		ov = (int) zval_is_true(arg4);
+		ov = (int) zend_is_true(arg4);
 		goto ipv6_loop_hops;
 	case IPV6_MULTICAST_HOPS:
 		convert_to_long(arg4);
diff --git a/ext/standard/head.c b/ext/standard/head.c
@@ -213,13 +213,13 @@ static zend_result php_head_parse_cookie_options_array(HashTable *options, zend_
 		} else if (zend_string_equals_literal_ci(key, "domain")) {
 			*domain = zval_get_string(value);
 		} else if (zend_string_equals_literal_ci(key, "secure")) {
-			*secure = zval_is_true(value);
+			*secure = zend_is_true(value);
 		} else if (zend_string_equals_literal_ci(key, "httponly")) {
-			*httponly = zval_is_true(value);
+			*httponly = zend_is_true(value);
 		} else if (zend_string_equals_literal_ci(key, "samesite")) {
 			*samesite = zval_get_string(value);
 		} else if (zend_string_equals_literal_ci(key, "partitioned")) {
-			*partitioned = zval_is_true(value);
+			*partitioned = zend_is_true(value);
 		} else {
 			zend_value_error("%s(): option \"%s\" is invalid", get_active_function_name(), ZSTR_VAL(key));
 			return FAILURE;
diff --git a/ext/standard/http_fopen_wrapper.c b/ext/standard/http_fopen_wrapper.c
@@ -276,7 +276,7 @@ static zend_string *php_stream_http_response_headers_parse(php_stream_wrapper *w
 	if (!strncasecmp(last_header_line, "Location:", sizeof("Location:")-1)) {
 		/* Check if the location should be followed. */
 		if (context && (tmpzval = php_stream_context_get_option(context, "http", "follow_location")) != NULL) {
-			header_info->follow_location = zval_is_true(tmpzval);
+			header_info->follow_location = zend_is_true(tmpzval);
 		} else if (!((response_code >= 300 && response_code < 304)
 				|| 307 == response_code || 308 == response_code)) {
 			/* The redirection should not be automatic if follow_location is not set and
diff --git a/main/streams/userspace.c b/main/streams/userspace.c
@@ -346,7 +346,7 @@ static php_stream *user_wrapper_opener(php_stream_wrapper *wrapper, const char *
 		zval_ptr_dtor(&args[3]);
 		goto end;
 	}
-	if (zval_is_true(&zretval)) {
+	if (zend_is_true(&zretval)) {
 		/* the stream is now open! */
 		stream = php_stream_alloc_rel(&php_stream_userspace_ops, us, 0, mode);
 
@@ -430,7 +430,7 @@ static php_stream *user_wrapper_opendir(php_stream_wrapper *wrapper, const char
 		goto end;
 	}
 
-	if (zval_is_true(&zretval)) {
+	if (zend_is_true(&zretval)) {
 		/* the stream is now open! */
 		stream = php_stream_alloc_rel(&php_stream_userspace_dir_ops, us, 0, mode);
 
@@ -672,7 +672,7 @@ static ssize_t php_userstreamop_read(php_stream *stream, char *buf, size_t count
 		goto err;
 	}
 
-	if (zval_is_true(&retval)) {
+	if (zend_is_true(&retval)) {
 		stream->eof = 1;
 	}
 	zval_ptr_dtor(&retval);
@@ -720,7 +720,7 @@ static int php_userstreamop_flush(php_stream *stream)
 	zend_result call_result = zend_call_method_if_exists(Z_OBJ(us->object), func_name, &retval, 0, NULL);
 	zend_string_release_ex(func_name, false);
 
-	int ret = call_result == SUCCESS && Z_TYPE(retval) != IS_UNDEF && zval_is_true(&retval) ? 0 : -1;
+	int ret = call_result == SUCCESS && Z_TYPE(retval) != IS_UNDEF && zend_is_true(&retval) ? 0 : -1;
 
 	zval_ptr_dtor(&retval);
 
@@ -755,7 +755,7 @@ static int php_userstreamop_seek(php_stream *stream, zend_off_t offset, int when
 
 		ret = -1;
 		goto out;
-	} else if (call_result == SUCCESS && Z_TYPE(retval) != IS_UNDEF && zval_is_true(&retval)) {
+	} else if (call_result == SUCCESS && Z_TYPE(retval) != IS_UNDEF && zend_is_true(&retval)) {
 		ret = 0;
 	} else {
 		ret = -1;
@@ -1090,7 +1090,7 @@ static int user_wrapper_unlink(php_stream_wrapper *wrapper, const char *url, int
 	} else if (Z_TYPE(zretval) == IS_FALSE || Z_TYPE(zretval) == IS_TRUE) {
 		ret = Z_TYPE(zretval) == IS_TRUE;
 	}
-	// TODO: Warn on invalid return type, or use zval_is_true()?
+	// TODO: Warn on invalid return type, or use zend_is_true()?
 
 	zval_ptr_dtor(&zretval);
 
@@ -1128,7 +1128,7 @@ static int user_wrapper_rename(php_stream_wrapper *wrapper, const char *url_from
 	} else if (Z_TYPE(zretval) == IS_FALSE || Z_TYPE(zretval) == IS_TRUE) {
 		ret = Z_TYPE(zretval) == IS_TRUE;
 	}
-	// TODO: Warn on invalid return type, or use zval_is_true()?
+	// TODO: Warn on invalid return type, or use zend_is_true()?
 
 	zval_ptr_dtor(&zretval);
 
@@ -1166,7 +1166,7 @@ static int user_wrapper_mkdir(php_stream_wrapper *wrapper, const char *url, int
 	} else if (Z_TYPE(zretval) == IS_FALSE || Z_TYPE(zretval) == IS_TRUE) {
 		ret = Z_TYPE(zretval) == IS_TRUE;
 	}
-	// TODO: Warn on invalid return type, or use zval_is_true()?
+	// TODO: Warn on invalid return type, or use zend_is_true()?
 
 	zval_ptr_dtor(&zretval);
 
@@ -1203,7 +1203,7 @@ static int user_wrapper_rmdir(php_stream_wrapper *wrapper, const char *url,
 	} else if (Z_TYPE(zretval) == IS_FALSE || Z_TYPE(zretval) == IS_TRUE) {
 		ret = Z_TYPE(zretval) == IS_TRUE;
 	}
-	// TODO: Warn on invalid return type, or use zval_is_true()?
+	// TODO: Warn on invalid return type, or use zend_is_true()?
 
 	zval_ptr_dtor(&zretval);
 
@@ -1265,7 +1265,7 @@ static int user_wrapper_metadata(php_stream_wrapper *wrapper, const char *url, i
 	} else if (Z_TYPE(zretval) == IS_FALSE || Z_TYPE(zretval) == IS_TRUE) {
 		ret = Z_TYPE(zretval) == IS_TRUE;
 	}
-	// TODO: Warn on invalid return type, or use zval_is_true()?
+	// TODO: Warn on invalid return type, or use zend_is_true()?
 
 	zval_ptr_dtor(&zretval);
 

Original file line number	Diff line number	Diff line change
`@@ -12062,7 +12062,7 @@ bool zend_try_ct_eval_cast(zval result, uint32_t type, zval op1)`
`12062`	`12062`	`}`
`12063`	`12063`	`switch (type) {`
`12064`	`12064`	`case _IS_BOOL:`
`12065`		`- ZVAL_BOOL(result, zval_is_true(op1));`
	`12065`	`+ ZVAL_BOOL(result, zend_is_true(op1));`
`12066`	`12066`	`return true;`
`12067`	`12067`	`case IS_LONG:`
`12068`	`12068`	`if (Z_TYPE_P(op1) == IS_DOUBLE && !ZEND_DOUBLE_FITS_LONG(Z_DVAL_P((op1)))) {`
Original file line number	Diff line number	Diff line change
`@@ -1593,7 +1593,7 @@ ZEND_API zend_result ZEND_FASTCALL boolean_xor_function(zval result, zval op1,`
`1593`	`1593`	`}`
`1594`	`1594`	`}`
`1595`	`1595`	`ZEND_TRY_BINARY_OP1_OBJECT_OPERATION(ZEND_BOOL_XOR);`
`1596`		`- op1_val = zval_is_true(op1);`
	`1596`	`+ op1_val = zend_is_true(op1);`
`1597`	`1597`	`}`
`1598`	`1598`	`} while (0);`
`1599`	`1599`	`do {`
`@@ -1613,7 +1613,7 @@ ZEND_API zend_result ZEND_FASTCALL boolean_xor_function(zval result, zval op1,`
`1613`	`1613`	`}`
`1614`	`1614`	`}`
`1615`	`1615`	`ZEND_TRY_BINARY_OP2_OBJECT_OPERATION(ZEND_BOOL_XOR);`
`1616`		`- op2_val = zval_is_true(op2);`
	`1616`	`+ op2_val = zend_is_true(op2);`
`1617`	`1617`	`}`
`1618`	`1618`	`} while (0);`
`1619`	`1619`
`@@ -1641,7 +1641,7 @@ ZEND_API zend_result ZEND_FASTCALL boolean_not_function(zval result, zval op1)`
`1641`	`1641`	`}`
`1642`	`1642`	`ZEND_TRY_UNARY_OBJECT_OPERATION(ZEND_BOOL_NOT);`
`1643`	`1643`
`1644`		`- ZVAL_BOOL(result, !zval_is_true(op1));`
	`1644`	`+ ZVAL_BOOL(result, !zend_is_true(op1));`
`1645`	`1645`	`}`
`1646`	`1646`	`return SUCCESS;`
`1647`	`1647`	`}`
`@@ -2433,13 +2433,13 @@ ZEND_API int ZEND_FASTCALL zend_compare(zval op1, zval op2) /* {{{ */`
`2433`	`2433`	`converted = true;`
`2434`	`2434`	`}`
`2435`	`2435`	`} else if (Z_TYPE_P(op1) < IS_TRUE) {`
`2436`		`- return zval_is_true(op2) ? -1 : 0;`
	`2436`	`+ return zend_is_true(op2) ? -1 : 0;`
`2437`	`2437`	`} else if (Z_TYPE_P(op1) == IS_TRUE) {`
`2438`		`- return zval_is_true(op2) ? 0 : 1;`
	`2438`	`+ return zend_is_true(op2) ? 0 : 1;`
`2439`	`2439`	`} else if (Z_TYPE_P(op2) < IS_TRUE) {`
`2440`		`- return zval_is_true(op1) ? 1 : 0;`
	`2440`	`+ return zend_is_true(op1) ? 1 : 0;`
`2441`	`2441`	`} else if (Z_TYPE_P(op2) == IS_TRUE) {`
`2442`		`- return zval_is_true(op1) ? 0 : -1;`
	`2442`	`+ return zend_is_true(op1) ? 0 : -1;`
`2443`	`2443`	`} else {`
`2444`	`2444`	`op1 = _zendi_convert_scalar_to_number_silent(op1, &op1_copy);`
`2445`	`2445`	`op2 = _zendi_convert_scalar_to_number_silent(op2, &op2_copy);`
Original file line number	Diff line number	Diff line change
`@@ -404,9 +404,6 @@ static zend_always_inline bool try_convert_to_string(zval *op) {`
`404`	`404`	`ZEND_API bool ZEND_FASTCALL zend_is_true(const zval *op);`
`405`	`405`	`ZEND_API bool ZEND_FASTCALL zend_object_is_true(const zval *op);`
`406`	`406`
`407`		`-#define zval_is_true(op) \`
`408`		`- zend_is_true(op)`
`409`		`-`
`410`	`407`	`static zend_always_inline bool i_zend_is_true(const zval *op)`
`411`	`408`	`{`
`412`	`409`	`bool result = 0;`
Original file line number	Diff line number	Diff line change
`@@ -316,7 +316,7 @@ static int mysqli_object_has_property(zend_object object, zend_string name, in`
`316`	`316`	`zval rv;`
`317`	`317`	`zval *value = mysqli_read_property(object, name, BP_VAR_IS, cache_slot, &rv);`
`318`	`318`	`if (value != &EG(uninitialized_zval)) {`
`319`		`- has_property = zval_is_true(value);`
	`319`	`+ has_property = zend_is_true(value);`
`320`	`320`	`zval_ptr_dtor(value);`
`321`	`321`	`}`
`322`	`322`	`break;`
Original file line number	Diff line number	Diff line change
`@@ -4967,7 +4967,7 @@ PHP_PGSQL_API zend_result php_pgsql_convert(PGconn pg_link, const zend_string `
`4967`	`4967`	`break; /* break out for() */`
`4968`	`4968`	`}`
`4969`	`4969`
`4970`		`- if (zval_is_true(is_enum)) {`
	`4970`	`+ if (zend_is_true(is_enum)) {`
`4971`	`4971`	`/* enums need to be treated like strings */`
`4972`	`4972`	`data_type = PG_TEXT;`
`4973`	`4973`	`} else {`