Refactor code and update documentation

Author: arefbehboudi

spring-boot-project/spring-boot/src/main/java/org/springframework/boot/jdbc/DataSourceBuilder.java

@@ -698,7 +698,6 @@ private void setDriverClass(ComboPooledDataSource dataSource, String driverClass
 	 */
 	private static class SimpleDataSourceProperties extends MappedDataSourceProperties<SimpleDriverDataSource> {
 
-		@SuppressWarnings("unchecked")
 		SimpleDataSourceProperties() {
 			add(DataSourceProperty.URL, SimpleDriverDataSource::getUrl, SimpleDriverDataSource::setUrl);
 			add(DataSourceProperty.DRIVER_CLASS_NAME, Class.class, (dataSource) -> dataSource.getDriver().getClass(),

spring-boot-project/spring-boot/src/main/java/org/springframework/boot/json/JsonValueWriter.java

@@ -106,8 +106,8 @@ else if (value instanceof Map<?, ?> map) {
 		else if (value instanceof Number) {
 			append(value.toString());
 		}
-		else if (value instanceof Boolean) {
-			append(Boolean.TRUE.equals(value) ? "true" : "false");
+		else if (value instanceof Boolean bool) {
+			append(bool ? "true" : "false");
 		}
 		else {
 			writeString(value);

spring-boot-project/spring-boot/src/main/java/org/springframework/boot/logging/log4j2/Log4J2LoggingSystem.java

@@ -238,9 +238,7 @@ protected void loadConfiguration(LoggingInitializationContext initializationCont
 
 	private void load(LoggingInitializationContext initializationContext, String location, LogFile logFile) {
 		List<String> overrides = getOverrides(initializationContext);
-		if (initializationContext != null) {
-			applySystemProperties(initializationContext.getEnvironment(), logFile);
-		}
+		applySystemProperties(initializationContext.getEnvironment(), logFile);
 		loadConfiguration(location, logFile, overrides);
 	}
 

spring-boot-project/spring-boot/src/main/java/org/springframework/boot/rsocket/context/RSocketPortInfoApplicationContextInitializer.java

@@ -51,49 +51,43 @@ public void initialize(ConfigurableApplicationContext applicationContext) {
 		applicationContext.addApplicationListener(new Listener(applicationContext));
 	}
 
-	private static class Listener implements ApplicationListener<RSocketServerInitializedEvent> {
+	private record Listener(ConfigurableApplicationContext applicationContext) implements ApplicationListener<RSocketServerInitializedEvent> {
 
-		private static final String PROPERTY_NAME = "local.rsocket.server.port";
+			private static final String PROPERTY_NAME = "local.rsocket.server.port";
 
-		private static final String PROPERTY_SOURCE_NAME = "server.ports";
-
-		private final ConfigurableApplicationContext applicationContext;
-
-		Listener(ConfigurableApplicationContext applicationContext) {
-			this.applicationContext = applicationContext;
-		}
+			private static final String PROPERTY_SOURCE_NAME = "server.ports";
 
 		@Override
-		public void onApplicationEvent(RSocketServerInitializedEvent event) {
-			if (event.getServer().address() != null) {
-				setPortProperty(this.applicationContext, event.getServer().address().getPort());
+			public void onApplicationEvent(RSocketServerInitializedEvent event) {
+				if (event.getServer().address() != null) {
+					setPortProperty(this.applicationContext, event.getServer().address().getPort());
+				}
 			}
-		}
 
-		private void setPortProperty(ApplicationContext context, int port) {
-			if (context instanceof ConfigurableApplicationContext configurableContext) {
-				setPortProperty(configurableContext.getEnvironment(), port);
+			private void setPortProperty(ApplicationContext context, int port) {
+				if (context instanceof ConfigurableApplicationContext configurableContext) {
+					setPortProperty(configurableContext.getEnvironment(), port);
+				}
+				if (context.getParent() != null) {
+					setPortProperty(context.getParent(), port);
+				}
 			}
-			if (context.getParent() != null) {
-				setPortProperty(context.getParent(), port);
+
+			private void setPortProperty(ConfigurableEnvironment environment, int port) {
+				MutablePropertySources sources = environment.getPropertySources();
+				PropertySource<?> source = sources.get(PROPERTY_SOURCE_NAME);
+				if (source == null) {
+					source = new MapPropertySource(PROPERTY_SOURCE_NAME, new HashMap<>());
+					sources.addFirst(source);
+				}
+				setPortProperty(port, source);
 			}
-		}
 
-		private void setPortProperty(ConfigurableEnvironment environment, int port) {
-			MutablePropertySources sources = environment.getPropertySources();
-			PropertySource<?> source = sources.get(PROPERTY_SOURCE_NAME);
-			if (source == null) {
-				source = new MapPropertySource(PROPERTY_SOURCE_NAME, new HashMap<>());
-				sources.addFirst(source);
+			@SuppressWarnings("unchecked")
+			private void setPortProperty(int port, PropertySource<?> source) {
+				((Map<String, Object>) source.getSource()).put(PROPERTY_NAME, port);
 			}
-			setPortProperty(port, source);
-		}
 
-		@SuppressWarnings("unchecked")
-		private void setPortProperty(int port, PropertySource<?> source) {
-			((Map<String, Object>) source.getSource()).put(PROPERTY_NAME, port);
 		}
 
-	}
-
 }

spring-boot-project/spring-boot/src/main/java/org/springframework/boot/security/reactive/ApplicationContextServerWebExchangeMatcher.java

@@ -29,7 +29,7 @@
 /**
  * {@link ApplicationContext} backed {@link ServerWebExchangeMatcher}. Can work directly
  * with the {@link ApplicationContext}, obtain an existing bean or
- * {@link AutowireCapableBeanFactory#createBean(Class, int, boolean) create a new bean}
+ * {@link AutowireCapableBeanFactory#createBean(Class) create a new bean}
  * that is autowired in the usual way.
  *
  * @param <C> the type of the context that the match method actually needs to use. Can be

spring-boot-project/spring-boot/src/main/java/org/springframework/boot/security/servlet/ApplicationContextRequestMatcher.java

@@ -30,7 +30,7 @@
 /**
  * {@link ApplicationContext} backed {@link RequestMatcher}. Can work directly with the
  * {@link ApplicationContext}, obtain an existing bean or
- * {@link AutowireCapableBeanFactory#createBean(Class, int, boolean) create a new bean}
+ * {@link AutowireCapableBeanFactory#createBean(Class) create a new bean}
  * that is autowired in the usual way.
  *
  * @param <C> the type of the context that the match method actually needs to use. Can be