Simplify processConstructorReflection: remove classReflection and classFound from return

Address review feedback: classFound is equivalent to checking
reflectionProvider->hasClass() and classReflection can be obtained
from the reflectionProvider at the call site, so
processConstructorReflection's signature doesn't need to change.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: phpstan-bot

src/Analyser/ExprHandler/NewHandler.php

@@ -86,13 +86,12 @@ public function processExpr(NodeScopeResolver $nodeScopeResolver, Stmt $stmt, Ex
 		$impurePoints = [];
 		$isAlwaysTerminating = false;
 		$normalizedExpr = $expr;
-		$classReflection = null;
-		$classFound = true;
+		$className = null;
 		$deferConstructorThrowPoints = false;
 		if ($expr->class instanceof Name) {
 			$className = $scope->resolveName($expr->class);
 
-			[$constructorReflection, $parametersAcceptor, $classReflection, $classFound, $constructorImpurePoints] = $this->processConstructorReflection($className, $expr, $scope);
+			[$constructorReflection, $parametersAcceptor, $constructorImpurePoints] = $this->processConstructorReflection($className, $expr, $scope);
 			$impurePoints = array_merge($impurePoints, $constructorImpurePoints);
 			$deferConstructorThrowPoints = true;
 
@@ -179,7 +178,7 @@ public function processExpr(NodeScopeResolver $nodeScopeResolver, Stmt $stmt, Ex
 			$throwPoints = array_merge($throwPoints, $additionalThrowPoints);
 
 			if ($className !== null) {
-				[$constructorReflection, $parametersAcceptor, $classReflection, $classFound, $constructorImpurePoints] = $this->processConstructorReflection($className, $expr, $scope);
+				[$constructorReflection, $parametersAcceptor, $constructorImpurePoints] = $this->processConstructorReflection($className, $expr, $scope);
 				$impurePoints = array_merge($impurePoints, $constructorImpurePoints);
 				$deferConstructorThrowPoints = true;
 			} else {
@@ -205,13 +204,14 @@ public function processExpr(NodeScopeResolver $nodeScopeResolver, Stmt $stmt, Ex
 		$impurePoints = array_merge($impurePoints, $argsResult->getImpurePoints());
 		$isAlwaysTerminating = $isAlwaysTerminating || $argsResult->isAlwaysTerminating();
 
-		if ($deferConstructorThrowPoints) {
-			if ($classReflection !== null && $constructorReflection !== null && $parametersAcceptor !== null) {
-				$constructorThrowPoint = $this->getConstructorThrowPoint($constructorReflection, $parametersAcceptor, $classReflection, $expr, new Name\FullyQualified($classReflection->getName()), $expr->getArgs(), $scope);
+		if ($deferConstructorThrowPoints && $className !== null) {
+			if ($constructorReflection !== null && $parametersAcceptor !== null && $this->reflectionProvider->hasClass($className)) {
+				$classReflection = $this->reflectionProvider->getClass($className);
+				$constructorThrowPoint = $this->getConstructorThrowPoint($constructorReflection, $parametersAcceptor, $classReflection, $expr, new Name\FullyQualified($className), $expr->getArgs(), $scope);
 				if ($constructorThrowPoint !== null) {
 					$throwPoints[] = $constructorThrowPoint;
 				}
-			} elseif (!$classFound) {
+			} elseif (!$this->reflectionProvider->hasClass($className)) {
 				$throwPoints[] = InternalThrowPoint::createImplicit($scope, $expr);
 			}
 		}
@@ -226,7 +226,7 @@ public function processExpr(NodeScopeResolver $nodeScopeResolver, Stmt $stmt, Ex
 	}
 
 	/**
-	 * @return array{?MethodReflection, ?ParametersAcceptor, ?ClassReflection, bool, ImpurePoint[]}
+	 * @return array{?MethodReflection, ?ParametersAcceptor, ImpurePoint[]}
 	 */
 	private function processConstructorReflection(string $className, New_ $expr, MutatingScope $scope): array
 	{
@@ -235,7 +235,6 @@ private function processConstructorReflection(string $className, New_ $expr, Mut
 		$impurePoints = [];
 
 		$classReflection = null;
-		$classFound = true;
 		if ($this->reflectionProvider->hasClass($className)) {
 			$classReflection = $this->reflectionProvider->getClass($className);
 			if ($classReflection->hasConstructor()) {
@@ -247,8 +246,6 @@ private function processConstructorReflection(string $className, New_ $expr, Mut
 					$constructorReflection->getNamedArgumentsVariants(),
 				);
 			}
-		} else {
-			$classFound = false;
 		}
 
 		if ($constructorReflection !== null) {
@@ -272,7 +269,7 @@ private function processConstructorReflection(string $className, New_ $expr, Mut
 			);
 		}
 
-		return [$constructorReflection, $parametersAcceptor, $classReflection, $classFound, $impurePoints];
+		return [$constructorReflection, $parametersAcceptor, $impurePoints];
 	}
 
 	/**