changes from feedback

Author: devnexen

ext/pcntl/pcntl.c

@@ -1637,7 +1637,7 @@ PHP_FUNCTION(pcntl_getqos_class)
 	{
 		// unlikely unless an external tool set the QOS class with a wrong value
 		PCNTL_G(last_error) = errno;
-		zend_value_error("invalid QOS class %u", qos_class);
+		zend_throw_error(NULL, "invalid QOS class %u", qos_class);
 		RETURN_THROWS();
 	}
 
@@ -1655,11 +1655,9 @@ PHP_FUNCTION(pcntl_setqos_class)
 	if (pthread_set_qos_class_self_np((qos_class_t)qos_class, 0) != 0)
 	{
 		PCNTL_G(last_error) = errno;
-		zend_argument_value_error(1, "must be between QOS_CLASS_USER_INTERACTIVE and QOS_CLASS_BLACKGROUND");
+		zend_argument_value_error(1, "must be one of QOS_CLASS_USER_INTERACTIVE, QOS_CLASS_USER_INITIATED, QOS_CLASS_DEFAULT, QOS_CLASS_UTILITY and QOS_CLASS_BACKGROUND");
 		RETURN_THROWS();
 	}
-
-	RETURN_TRUE;
 }
 #endif
 

ext/pcntl/pcntl.stub.php

@@ -1039,5 +1039,5 @@ function pcntl_getcpu(): int {}
 
 #ifdef HAVE_PTHREAD_SET_QOS_CLASS_SELF_NP
 function pcntl_getqos_class(): int {}
-function pcntl_setqos_class(int $qos_class = QOS_CLASS_DEFAULT): bool {}
+function pcntl_setqos_class(int $qos_class = QOS_CLASS_DEFAULT): void {}
 #endif

ext/pcntl/pcntl_arginfo.h

@@ -9,20 +9,18 @@ if (getenv('SKIP_REPEAT')) die("skip Not repeatable");
 ?>
 --FILE--
 <?php
-var_dump(pcntl_setqos_class(QOS_CLASS_DEFAULT));
+pcntl_setqos_class(QOS_CLASS_DEFAULT);
 var_dump(QOS_CLASS_DEFAULT === pcntl_getqos_class());
 
 try {
 	pcntl_setqos_class(QOS_CLASS_UNSPECIFIED);
 } catch (\ValueError $e) {
 	echo $e->getMessage() . PHP_EOL;
 }
-var_dump(pcntl_setqos_class(QOS_CLASS_BACKGROUND));
+pcntl_setqos_class(QOS_CLASS_BACKGROUND);
 var_dump(QOS_CLASS_BACKGROUND === pcntl_getqos_class());
 ?>
 --EXPECT--
 bool(true)
-bool(true)
-pcntl_setqos_class(): Argument #1 ($qos_class) must be between QOS_CLASS_USER_INTERACTIVE and QOS_CLASS_BLACKGROUND
-bool(true)
+pcntl_setqos_class(): Argument #1 ($qos_class) must be one of QOS_CLASS_USER_INTERACTIVE, QOS_CLASS_USER_INITIATED, QOS_CLASS_DEFAULT, QOS_CLASS_UTILITY and QOS_CLASS_BACKGROUND
 bool(true)