Commit e48af31c authored by Johnny Lim's avatar Johnny Lim Committed by Stephane Nicoll

Polish

See gh-23081
parent 4320b3a9
......@@ -57,7 +57,6 @@ public enum Include {
PRINCIPAL,
/**
*
* Include the remote address.
*/
REMOTE_ADDRESS,
......
......@@ -76,7 +76,7 @@ class SharedMetadataReaderFactoryContextInitializer
static class CachingMetadataReaderFactoryPostProcessor
implements BeanDefinitionRegistryPostProcessor, PriorityOrdered {
private ConfigurableApplicationContext context;
private final ConfigurableApplicationContext context;
CachingMetadataReaderFactoryPostProcessor(ConfigurableApplicationContext context) {
this.context = context;
......
......@@ -101,8 +101,8 @@ public class Repackager extends Packager {
public void repackage(File destination, Libraries libraries, LaunchScript launchScript, FileTime lastModifiedTime)
throws IOException {
Assert.isTrue(destination != null && !destination.isDirectory(), "Invalid destination");
getLayout(); // get layout early
if (lastModifiedTime != null && getLayout() instanceof War) {
Layout layout = getLayout(); // get layout early
if (lastModifiedTime != null && layout instanceof War) {
throw new IllegalStateException("Reproducible repackaging is not supported with war packaging");
}
destination = destination.getAbsoluteFile();
......
......@@ -25,7 +25,7 @@ import java.lang.annotation.Target;
import org.junit.jupiter.api.extension.ExtendWith;
/**
* Annotation used to fork the classpath. This can be helpful were neither
* Annotation used to fork the classpath. This can be helpful where neither
* {@link ClassPathExclusions} or {@link ClassPathOverrides} are needed, but just a copy
* of the classpath.
*
......
......@@ -259,7 +259,7 @@ public class ConfigFileApplicationListener implements EnvironmentPostProcessor,
*/
private static class PropertySourceOrderingPostProcessor implements BeanFactoryPostProcessor, Ordered {
private ConfigurableApplicationContext context;
private final ConfigurableApplicationContext context;
PropertySourceOrderingPostProcessor(ConfigurableApplicationContext context) {
this.context = context;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment