don't return null anymore

Author: mattdinthehouse

ext/reflection/php_reflection.c

@@ -5173,7 +5173,7 @@ ZEND_METHOD(ReflectionClass, newInstanceFromData)
 		if (!(constructor->common.fn_flags & ZEND_ACC_PUBLIC)) {
 			zend_throw_exception_ex(reflection_exception_ptr, 0, "Access to non-public constructor of class %s", ZSTR_VAL(ce->name));
 			zval_ptr_dtor(return_value);
-			RETURN_NULL();
+			RETURN_THROWS();
 		}
 
 		zend_call_known_function(

ext/reflection/php_reflection.stub.php

@@ -370,7 +370,7 @@ public function newInstanceWithoutConstructor(): object {}
     /** @tentative-return-type */
     public function newInstanceArgs(array $args = []): ?object {}
 
-    public function newInstanceFromData(array $data, array $args = []): ?object {}
+    public function newInstanceFromData(array $data, array $args = []): object {}
 
     public function newLazyGhost(callable $initializer, int $options = 0): object {}
 

ext/reflection/php_reflection_arginfo.h

@@ -338,7 +338,7 @@ Class [ <internal:Reflection> class ReflectionClass implements Stringable, Refle
         Parameter #0 [ <required> array $data ]
         Parameter #1 [ <optional> array $args = [] ]
       }
-      - Return [ ?object ]
+      - Return [ object ]
     }
 
     Method [ <internal:Reflection> public method newLazyGhost ] {