Unnecessary call to AccessResult::neutral() in Drupal\Core\Access\AccessManager::check()

Problem/Motivation

In the following code (Drupal\Core\Access\AccessManager::check(), line 130) AccessResult::neutral() is preloaded and stored in $result. 5 lines later another call to this method is made return AccessResult::neutral();. Returning $result does exactly the same and saves one method call.

$result = AccessResult::neutral();
if (!empty($checks)) {
  $arguments_resolver = $this->argumentsResolverFactory->getArgumentsResolver($route_match, $account, $request);

  if (!$checks) {
    return AccessResult::neutral();
  }

Proposed resolution

Return $result instead of AccessResult::neutral().

Remaining tasks

None

User interface changes

None

API changes

None

Data model changes

None