fixup! Address code review comments

Author: kocsismate

Zend/zend_closures.stub.php

@@ -17,9 +17,6 @@ public function bindTo(?object $newThis, $newScope = UNKNOWN): ?Closure {}
 
     public function call(object $newThis, mixed ...$arguments): mixed {}
 
-    /**
-     * callable is not a proper type due to bug #78770.
-     * @param callable $callback
-     */
+    /** @param callable $callback callable is not a proper type due to bug #78770. */
     public static function fromCallable($callback): Closure {}
 }

Zend/zend_closures_arginfo.h

@@ -1,5 +1,5 @@
 /* This is a generated file, edit the .stub.php file instead.
- * Stub hash: ed6150159a1c45891c2d148771c366fc3b1ef8f0 */
+ * Stub hash: 62198e96940fe0e86fe89601015c837aa5390e92 */
 
 ZEND_BEGIN_ARG_INFO_EX(arginfo_class_Closure___construct, 0, 0, 0)
 ZEND_END_ARG_INFO()

ext/mbstring/mbstring.stub.php

@@ -71,10 +71,6 @@ function mb_decode_mimeheader(string $string): string {}
 
 function mb_convert_kana(string $str, string $option = "KV", ?string $encoding = null): string {}
 
-/**
- * @param object|string|array $var
- * @param object|string|array $vars
- */
 function mb_convert_variables(string $to, array|string $from, mixed &$var, mixed &...$vars): string|false {}
 
 function mb_encode_numericentity(string $string, array $convmap, ?string $encoding = null, bool $is_hex = false): string {}

ext/mbstring/mbstring_arginfo.h

@@ -1,5 +1,5 @@
 /* This is a generated file, edit the .stub.php file instead.
- * Stub hash: 615e21b594825ea0d71065f7816e77969d1aff30 */
+ * Stub hash: 5ad8a8cf20eeae59713d19135ecccbee243754eb */
 
 ZEND_BEGIN_ARG_WITH_RETURN_TYPE_MASK_EX(arginfo_mb_language, 0, 0, MAY_BE_STRING|MAY_BE_BOOL)
 	ZEND_ARG_TYPE_INFO_WITH_DEFAULT_VALUE(0, language, IS_STRING, 1, "null")

ext/reflection/php_reflection.c

@@ -3100,7 +3100,11 @@ ZEND_METHOD(ReflectionMethod, getClosure)
 
 	if (mptr->common.fn_flags & ZEND_ACC_STATIC)  {
 		zend_create_fake_closure(return_value, mptr, mptr->common.scope, mptr->common.scope, NULL);
-	} else if (obj) {
+	} else {
+		if (!obj) {
+			zend_argument_value_error(1, "cannot be null for non-static methods");
+		}
+
 		if (!instanceof_function(Z_OBJCE_P(obj), mptr->common.scope)) {
 			_DO_THROW("Given object is not an instance of the class this method was declared in");
 			RETURN_THROWS();
@@ -3114,8 +3118,6 @@ ZEND_METHOD(ReflectionMethod, getClosure)
 		} else {
 			zend_create_fake_closure(return_value, mptr, mptr->common.scope, Z_OBJCE_P(obj), obj);
 		}
-	} else {
-		zend_argument_value_error(1, "cannot be null for non-static methods");
 	}
 }
 /* }}} */