ext/standard/basic_functions.c: return bool instead of int for ignore_user_abort()

The INI setting is stored and displayed as a bool already.

Author: Girgias

ext/standard/basic_functions.c

@@ -2113,22 +2113,22 @@ PHP_FUNCTION(ignore_user_abort)
 {
 	bool arg = 0;
 	bool arg_is_null = 1;
-	int old_setting;
+	bool old_setting;
 
 	ZEND_PARSE_PARAMETERS_START(0, 1)
 		Z_PARAM_OPTIONAL
 		Z_PARAM_BOOL_OR_NULL(arg, arg_is_null)
 	ZEND_PARSE_PARAMETERS_END();
 
-	old_setting = (unsigned short)PG(ignore_user_abort);
+	old_setting = PG(ignore_user_abort);
 
 	if (!arg_is_null) {
 		zend_string *key = ZSTR_INIT_LITERAL("ignore_user_abort", 0);
 		zend_alter_ini_entry_chars(key, arg ? "1" : "0", 1, PHP_INI_USER, PHP_INI_STAGE_RUNTIME);
 		zend_string_release_ex(key, 0);
 	}
 
-	RETURN_LONG(old_setting);
+	RETURN_BOOL(old_setting);
 }
 /* }}} */
 

ext/standard/basic_functions.stub.php

@@ -2039,7 +2039,7 @@ function connection_aborted(): int {}
 
 function connection_status(): int {}
 
-function ignore_user_abort(?bool $enable = null): int {}
+function ignore_user_abort(?bool $enable = null): bool {}
 
 #ifdef HAVE_GETSERVBYNAME
 function getservbyname(string $service, string $protocol): int|false {}

ext/standard/basic_functions_arginfo.h

@@ -14,9 +14,9 @@ var_dump(ini_get("ignore_user_abort"));
 
 ?>
 --EXPECT--
-int(0)
-int(1)
+bool(false)
+bool(true)
 string(1) "1"
-int(1)
-int(0)
+bool(true)
+bool(false)
 string(1) "0"