Skip to content

Commit

Permalink
Simplify DefaultSingletonBeanRegistry.isDependent()
Browse files Browse the repository at this point in the history
Move `alreadySeen` handling out of for-loop.

Closes gh-30839
  • Loading branch information
bnbakp0582 authored and sbrannen committed Jul 9, 2023
1 parent b3de1b8 commit fb4ad2f
Showing 1 changed file with 4 additions and 4 deletions.
Expand Up @@ -453,11 +453,11 @@ private boolean isDependent(String beanName, String dependentBeanName, @Nullable
if (dependentBeans.contains(dependentBeanName)) {
return true;
}
if (alreadySeen == null) {
alreadySeen = new HashSet<>();
}
alreadySeen.add(beanName);
for (String transitiveDependency : dependentBeans) {
if (alreadySeen == null) {
alreadySeen = new HashSet<>();
}
alreadySeen.add(beanName);
if (isDependent(transitiveDependency, dependentBeanName, alreadySeen)) {
return true;
}
Expand Down

0 comments on commit fb4ad2f

Please sign in to comment.