Treat "static" as class type in reflection

Author: nikic

ext/reflection/php_reflection.c

@@ -2909,7 +2909,9 @@ ZEND_METHOD(reflection_named_type, isBuiltin)
 	}
 	GET_REFLECTION_OBJECT_PTR(param);
 
-	RETVAL_BOOL(ZEND_TYPE_IS_ONLY_MASK(param->type));
+	/* Treat "static" as a class type for the purposes of reflection. */
+	RETVAL_BOOL(ZEND_TYPE_IS_ONLY_MASK(param->type)
+		&& !(ZEND_TYPE_FULL_MASK(param->type) & MAY_BE_STATIC));
 }
 /* }}} */
 

ext/reflection/tests/static_type.phpt

@@ -11,13 +11,12 @@ class A {
 
 $rm = new ReflectionMethod(A::class, 'test');
 $rt = $rm->getReturnType();
-// TODO: Should it be considered a builtin or not?
 var_dump($rt->isBuiltin());
 var_dump($rt->getName());
 var_dump((string) $rt);
 
 ?>
 --EXPECT--
-bool(true)
+bool(false)
 string(6) "static"
 string(6) "static"