Skip to content

Commit c2b2f30

Browse files
committed
Polishing
1 parent 239a7cd commit c2b2f30

File tree

1 file changed

+11
-7
lines changed

1 file changed

+11
-7
lines changed

spring-context/src/main/java/org/springframework/context/annotation/ConfigurationClassParser.java

Lines changed: 11 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -85,8 +85,6 @@ class ConfigurationClassParser {
8585

8686
private final ProblemReporter problemReporter;
8787

88-
private final ImportStack importStack = new ImportStack();
89-
9088
private final Environment environment;
9189

9290
private final ResourceLoader resourceLoader;
@@ -101,6 +99,8 @@ class ConfigurationClassParser {
10199

102100
private final Stack<PropertySource<?>> propertySources = new Stack<PropertySource<?>>();
103101

102+
private final ImportStack importStack = new ImportStack();
103+
104104

105105
/**
106106
* Create a new {@link ConfigurationClassParser} instance that will be used
@@ -377,23 +377,27 @@ private void processImport(ConfigurationClass configClass, Collection<?> classes
377377
AnnotationMetadata importingClassMetadata = configClass.getMetadata();
378378
try {
379379
for (Object candidate : classesToImport) {
380-
Object candidateToCheck = (candidate instanceof Class ? (Class) candidate : this.metadataReaderFactory.getMetadataReader((String) candidate));
380+
Object candidateToCheck = (candidate instanceof Class ? (Class) candidate :
381+
this.metadataReaderFactory.getMetadataReader((String) candidate));
381382
if (checkAssignability(ImportSelector.class, candidateToCheck)) {
382383
// the candidate class is an ImportSelector -> delegate to it to determine imports
383-
Class<?> candidateClass = (candidate instanceof Class ? (Class) candidate : this.resourceLoader.getClassLoader().loadClass((String) candidate));
384+
Class<?> candidateClass = (candidate instanceof Class ? (Class) candidate :
385+
this.resourceLoader.getClassLoader().loadClass((String) candidate));
384386
ImportSelector selector = BeanUtils.instantiateClass(candidateClass, ImportSelector.class);
385387
processImport(configClass, Arrays.asList(selector.selectImports(importingClassMetadata)), false);
386388
}
387389
else if (checkAssignability(ImportBeanDefinitionRegistrar.class, candidateToCheck)) {
388390
// the candidate class is an ImportBeanDefinitionRegistrar -> delegate to it to register additional bean definitions
389-
Class<?> candidateClass = (candidate instanceof Class ? (Class) candidate : this.resourceLoader.getClassLoader().loadClass((String) candidate));
391+
Class<?> candidateClass = (candidate instanceof Class ? (Class) candidate :
392+
this.resourceLoader.getClassLoader().loadClass((String) candidate));
390393
ImportBeanDefinitionRegistrar registrar = BeanUtils.instantiateClass(candidateClass, ImportBeanDefinitionRegistrar.class);
391394
invokeAwareMethods(registrar);
392395
registrar.registerBeanDefinitions(importingClassMetadata, this.registry);
393396
}
394397
else {
395398
// candidate class not an ImportSelector or ImportBeanDefinitionRegistrar -> process it as a @Configuration class
396-
this.importStack.registerImport(importingClassMetadata.getClassName(), (candidate instanceof Class ? ((Class) candidate).getName() : (String) candidate));
399+
this.importStack.registerImport(importingClassMetadata.getClassName(),
400+
(candidate instanceof Class ? ((Class) candidate).getName() : (String) candidate));
397401
processConfigurationClass(candidateToCheck instanceof Class ? new ConfigurationClass((Class) candidateToCheck, true) :
398402
new ConfigurationClass((MetadataReader) candidateToCheck, true));
399403
}
@@ -457,7 +461,7 @@ public Stack<PropertySource<?>> getPropertySources() {
457461
return this.propertySources;
458462
}
459463

460-
public ImportRegistry getImportRegistry() {
464+
ImportRegistry getImportRegistry() {
461465
return this.importStack;
462466
}
463467

0 commit comments

Comments
 (0)