Refactor iterator of Map with Java8's Map.forEach

diguage · snicoll · commit 1ef5f61ab2b4 · 2017-06-13T16:06:20.000+02:00
See gh-1459
diff --git a/spring-context-indexer/src/main/java/org/springframework/context/index/PropertiesMarshaller.java b/spring-context-indexer/src/main/java/org/springframework/context/index/PropertiesMarshaller.java
@@ -21,7 +21,6 @@
 import java.io.OutputStream;
 import java.util.Arrays;
 import java.util.HashSet;
-import java.util.Map;
 import java.util.Properties;
 import java.util.Set;
 
@@ -43,11 +42,10 @@ public static CandidateComponentsMetadata read(InputStream in) throws IOExceptio
 		CandidateComponentsMetadata result = new CandidateComponentsMetadata();
 		Properties props = new Properties();
 		props.load(in);
-		for (Map.Entry<Object, Object> entry : props.entrySet()) {
-			String type = (String) entry.getKey();
-			Set<String> candidates = new HashSet<>(Arrays.asList(((String) entry.getValue()).split(",")));
-			result.add(new ItemMetadata(type, candidates));
-		}
+		props.forEach((type, value) -> {
+			Set<String> candidates = new HashSet<>(Arrays.asList(((String) value).split(",")));
+			result.add(new ItemMetadata((String) type, candidates));
+		});
 		return result;
 	}
 
diff --git a/spring-context-support/src/main/java/org/springframework/cache/caffeine/CaffeineCacheManager.java b/spring-context-support/src/main/java/org/springframework/cache/caffeine/CaffeineCacheManager.java
@@ -19,7 +19,6 @@
 import java.util.Arrays;
 import java.util.Collection;
 import java.util.Collections;
-import java.util.Map;
 import java.util.concurrent.ConcurrentHashMap;
 import java.util.concurrent.ConcurrentMap;
 
@@ -223,9 +222,7 @@ private void doSetCaffeine(Caffeine<Object, Object> cacheBuilder) {
 	 * Create the known caches again with the current state of this manager.
 	 */
 	private void refreshKnownCaches() {
-		for (Map.Entry<String, Cache> entry : this.cacheMap.entrySet()) {
-			entry.setValue(createCaffeineCache(entry.getKey()));
-		}
+		this.cacheMap.forEach((key, value) -> createCaffeineCache(key));
 	}
 
 }
diff --git a/spring-jdbc/src/main/java/org/springframework/jdbc/core/namedparam/MapSqlParameterSource.java b/spring-jdbc/src/main/java/org/springframework/jdbc/core/namedparam/MapSqlParameterSource.java
@@ -131,13 +131,12 @@ public MapSqlParameterSource addValue(String paramName, Object value, int sqlTyp
 	 */
 	public MapSqlParameterSource addValues(@Nullable Map<String, ?> values) {
 		if (values != null) {
-			for (Map.Entry<String, ?> entry : values.entrySet()) {
-				this.values.put(entry.getKey(), entry.getValue());
-				if (entry.getValue() instanceof SqlParameterValue) {
-					SqlParameterValue value = (SqlParameterValue) entry.getValue();
-					registerSqlType(entry.getKey(), value.getSqlType());
+			values.forEach((key, value) -> {
+				this.values.put(key, value);
+				if (value instanceof SqlParameterValue) {
+					registerSqlType(key, ((SqlParameterValue) value).getSqlType());
 				}
-			}
+			});
 		}
 		return this;
 	}
diff --git a/spring-jdbc/src/main/java/org/springframework/jdbc/datasource/lookup/AbstractRoutingDataSource.java b/spring-jdbc/src/main/java/org/springframework/jdbc/datasource/lookup/AbstractRoutingDataSource.java
@@ -114,11 +114,11 @@ public void afterPropertiesSet() {
 			throw new IllegalArgumentException("Property 'targetDataSources' is required");
 		}
 		this.resolvedDataSources = new HashMap<>(this.targetDataSources.size());
-		for (Map.Entry<Object, Object> entry : this.targetDataSources.entrySet()) {
-			Object lookupKey = resolveSpecifiedLookupKey(entry.getKey());
-			DataSource dataSource = resolveSpecifiedDataSource(entry.getValue());
+		this.targetDataSources.forEach((key, value) -> {
+			Object lookupKey = resolveSpecifiedLookupKey(key);
+			DataSource dataSource = resolveSpecifiedDataSource(value);
 			this.resolvedDataSources.put(lookupKey, dataSource);
-		}
+		});
 		if (this.defaultTargetDataSource != null) {
 			this.resolvedDefaultDataSource = resolveSpecifiedDataSource(this.defaultTargetDataSource);
 		}
diff --git a/spring-jms/src/main/java/org/springframework/jms/annotation/JmsListenerAnnotationBeanPostProcessor.java b/spring-jms/src/main/java/org/springframework/jms/annotation/JmsListenerAnnotationBeanPostProcessor.java
@@ -225,12 +225,8 @@ public Object postProcessAfterInitialization(final Object bean, String beanName)
 			}
 			else {
 				// Non-empty set of methods
-				for (Map.Entry<Method, Set<JmsListener>> entry : annotatedMethods.entrySet()) {
-					Method method = entry.getKey();
-					for (JmsListener listener : entry.getValue()) {
-						processJmsListener(listener, method, bean);
-					}
-				}
+				annotatedMethods.forEach((method, listeners) ->
+						listeners.forEach(listener -> processJmsListener(listener, method, bean)));
 				if (logger.isDebugEnabled()) {
 					logger.debug(annotatedMethods.size() + " @JmsListener methods processed on bean '" + beanName +
 							"': " + annotatedMethods);
diff --git a/spring-jms/src/main/java/org/springframework/jms/support/converter/MappingJackson2MessageConverter.java b/spring-jms/src/main/java/org/springframework/jms/support/converter/MappingJackson2MessageConverter.java
@@ -158,12 +158,10 @@ public void setTypeIdPropertyName(String typeIdPropertyName) {
 	 */
 	public void setTypeIdMappings(Map<String, Class<?>> typeIdMappings) {
 		this.idClassMappings = new HashMap<>();
-		for (Map.Entry<String, Class<?>> entry : typeIdMappings.entrySet()) {
-			String id = entry.getKey();
-			Class<?> clazz = entry.getValue();
+		typeIdMappings.forEach((id, clazz) -> {
 			this.idClassMappings.put(id, clazz);
 			this.classIdMappings.put(clazz, id);
-		}
+		});
 	}
 
 	@Override
diff --git a/spring-messaging/src/main/java/org/springframework/messaging/MessageHeaders.java b/spring-messaging/src/main/java/org/springframework/messaging/MessageHeaders.java
@@ -149,11 +149,11 @@ else if (timestamp < 0) {
 	 */
 	private MessageHeaders(MessageHeaders original, Set<String> keysToIgnore) {
 		this.headers = new HashMap<>(original.headers.size() - keysToIgnore.size());
-		for (Map.Entry<String, Object> entry : original.headers.entrySet()) {
-			if (!keysToIgnore.contains(entry.getKey())) {
-				this.headers.put(entry.getKey(), entry.getValue());
+		original.headers.forEach((key, value) -> {
+			if (!keysToIgnore.contains(key)) {
+				this.headers.put(key, value);
 			}
-		}
+		});
 	}
 
 
@@ -276,11 +276,11 @@ public void clear() {
 
 	private void writeObject(ObjectOutputStream out) throws IOException {
 		Set<String> keysToIgnore = new HashSet<>();
-		for (Map.Entry<String, Object> entry : this.headers.entrySet()) {
-			if (!(entry.getValue() instanceof Serializable)) {
-				keysToIgnore.add(entry.getKey());
+		this.headers.forEach((key, value) -> {
+			if (!(value instanceof Serializable)) {
+				keysToIgnore.add(key);
 			}
-		}
+		});
 
 		if (keysToIgnore.isEmpty()) {
 			// All entries are serializable -> serialize the regular MessageHeaders instance
diff --git a/spring-messaging/src/main/java/org/springframework/messaging/handler/invocation/AbstractMethodMessageHandler.java b/spring-messaging/src/main/java/org/springframework/messaging/handler/invocation/AbstractMethodMessageHandler.java
@@ -288,9 +288,7 @@ protected final void detectHandlerMethods(final Object handler) {
 			if (logger.isDebugEnabled()) {
 				logger.debug(methods.size() + " message handler methods found on " + userType + ": " + methods);
 			}
-			for (Map.Entry<Method, T> entry : methods.entrySet()) {
-				registerHandlerMethod(handler, entry.getKey(), entry.getValue());
-			}
+			methods.forEach((key, value) -> registerHandlerMethod(handler, key, value));
 		}
 	}
 
diff --git a/spring-messaging/src/main/java/org/springframework/messaging/simp/stomp/StompHeaders.java b/spring-messaging/src/main/java/org/springframework/messaging/simp/stomp/StompHeaders.java
@@ -113,10 +113,7 @@ private StompHeaders(Map<String, List<String>> headers, boolean readOnly) {
 		Assert.notNull(headers, "'headers' must not be null");
 		if (readOnly) {
 			Map<String, List<String>> map = new LinkedMultiValueMap<>(headers.size());
-			for (Entry<String, List<String>> entry : headers.entrySet()) {
-				List<String> values = Collections.unmodifiableList(entry.getValue());
-				map.put(entry.getKey(), values);
-			}
+			headers.forEach((key, value) -> map.put(key, Collections.unmodifiableList(value)));
 			this.headers = Collections.unmodifiableMap(map);
 		}
 		else {
@@ -424,9 +421,7 @@ public void addAll(String headerName, List<? extends String> headerValues) {
 
 	@Override
 	public void addAll(MultiValueMap<String, String> values) {
-		for (Entry<String, List<String>> entry : values.entrySet()) {
-			addAll(entry.getKey(), entry.getValue());
-		}
+		values.forEach(this::addAll);
 	}
 
 	/**
@@ -446,17 +441,13 @@ public void set(String headerName, String headerValue) {
 
 	@Override
 	public void setAll(Map<String, String> values) {
-		for (Entry<String, String> entry : values.entrySet()) {
-			set(entry.getKey(), entry.getValue());
-		}
+		values.forEach(this::set);
 	}
 
 	@Override
 	public Map<String, String> toSingleValueMap() {
 		LinkedHashMap<String, String> singleValueMap = new LinkedHashMap<>(this.headers.size());
-		for (Entry<String, List<String>> entry : headers.entrySet()) {
-			singleValueMap.put(entry.getKey(), entry.getValue().get(0));
-		}
+		headers.forEach((key, value) -> singleValueMap.put(key, value.get(0)));
 		return singleValueMap;
 	}
 
diff --git a/spring-messaging/src/main/java/org/springframework/messaging/support/MessageHeaderAccessor.java b/spring-messaging/src/main/java/org/springframework/messaging/support/MessageHeaderAccessor.java
@@ -387,11 +387,11 @@ private List<String> getMatchingHeaderNames(String pattern, @Nullable Map<String
 	 */
 	public void copyHeaders(@Nullable Map<String, ?> headersToCopy) {
 		if (headersToCopy != null) {
-			for (Map.Entry<String, ?> entry : headersToCopy.entrySet()) {
-				if (!isReadOnly(entry.getKey())) {
-					setHeader(entry.getKey(), entry.getValue());
+			headersToCopy.forEach((key, value) -> {
+				if (!isReadOnly(key)) {
+					setHeader(key, value);
 				}
-			}
+			});
 		}
 	}
 
@@ -401,11 +401,11 @@ public void copyHeaders(@Nullable Map<String, ?> headersToCopy) {
 	 */
 	public void copyHeadersIfAbsent(@Nullable Map<String, ?> headersToCopy) {
 		if (headersToCopy != null) {
-			for (Map.Entry<String, ?> entry : headersToCopy.entrySet()) {
-				if (!isReadOnly(entry.getKey())) {
-					setHeaderIfAbsent(entry.getKey(), entry.getValue());
+			headersToCopy.forEach((key, value) -> {
+				if (!isReadOnly(key)) {
+					setHeaderIfAbsent(key, value);
 				}
-			}
+			});
 		}
 	}
 
diff --git a/spring-messaging/src/main/java/org/springframework/messaging/support/NativeMessageHeaderAccessor.java b/spring-messaging/src/main/java/org/springframework/messaging/support/NativeMessageHeaderAccessor.java
@@ -197,11 +197,7 @@ public void addNativeHeaders(@Nullable MultiValueMap<String, String> headers) {
 		if (headers == null) {
 			return;
 		}
-		for (Map.Entry<String, List<String>> headerEntry : headers.entrySet()) {
-			for (String value : headerEntry.getValue()) {
-				addNativeHeader(headerEntry.getKey(), value);
-			}
-		}
+		headers.forEach((key, values) -> values.forEach(value -> addNativeHeader(key, value)));
 	}
 
 	@Nullable
diff --git a/spring-orm/src/main/java/org/springframework/orm/jpa/AbstractEntityManagerFactoryBean.java b/spring-orm/src/main/java/org/springframework/orm/jpa/AbstractEntityManagerFactoryBean.java
@@ -325,11 +325,11 @@ public final void afterPropertiesSet() throws PersistenceException {
 			}
 			Map<String, ?> vendorPropertyMap = this.jpaVendorAdapter.getJpaPropertyMap();
 			if (vendorPropertyMap != null) {
-				for (Map.Entry<String, ?> entry : vendorPropertyMap.entrySet()) {
-					if (!this.jpaPropertyMap.containsKey(entry.getKey())) {
-						this.jpaPropertyMap.put(entry.getKey(), entry.getValue());
+				vendorPropertyMap.forEach((key, value) -> {
+					if (!this.jpaPropertyMap.containsKey(key)) {
+						this.jpaPropertyMap.put(key, value);
 					}
-				}
+				});
 			}
 			if (this.entityManagerFactoryInterface == null) {
 				this.entityManagerFactoryInterface = this.jpaVendorAdapter.getEntityManagerFactoryInterface();
diff --git a/spring-oxm/src/main/java/org/springframework/oxm/castor/CastorMarshaller.java b/spring-oxm/src/main/java/org/springframework/oxm/castor/CastorMarshaller.java
@@ -474,9 +474,7 @@ protected XMLContext createXMLContext(Resource[] mappingLocations, Class<?>[] ta
 			context.addPackages(targetPackages);
 		}
 		if (this.castorProperties != null) {
-			for (Map.Entry<String, String> property : this.castorProperties.entrySet()) {
-				context.setProperty(property.getKey(), property.getValue());
-			}
+			this.castorProperties.forEach(context::setProperty);
 		}
 		return context;
 	}
@@ -563,19 +561,13 @@ protected void customizeMarshaller(Marshaller marshaller) {
 		marshaller.setSchemaLocation(this.schemaLocation);
 		marshaller.setUseXSITypeAtRoot(this.useXSITypeAtRoot);
 		if (this.doctypes != null) {
-			for (Map.Entry<String, String> doctype : this.doctypes.entrySet()) {
-				marshaller.setDoctype(doctype.getKey(), doctype.getValue());
-			}
+			this.doctypes.forEach(marshaller::setDoctype);
 		}
 		if (this.processingInstructions != null) {
-			for (Map.Entry<String, String> processingInstruction : this.processingInstructions.entrySet()) {
-				marshaller.addProcessingInstruction(processingInstruction.getKey(), processingInstruction.getValue());
-			}
+			this.processingInstructions.forEach(marshaller::addProcessingInstruction);
 		}
 		if (this.namespaceMappings != null) {
-			for (Map.Entry<String, String> entry : this.namespaceMappings.entrySet()) {
-				marshaller.setNamespaceMapping(entry.getKey(), entry.getValue());
-			}
+			this.namespaceMappings.forEach(marshaller::setNamespaceMapping);
 		}
 	}
 
@@ -666,9 +658,7 @@ protected void customizeUnmarshaller(Unmarshaller unmarshaller) {
 		unmarshaller.setReuseObjects(this.reuseObjects);
 		unmarshaller.setClearCollections(this.clearCollections);
 		if (this.namespaceToPackageMapping != null) {
-			for (Map.Entry<String, String> mapping : this.namespaceToPackageMapping.entrySet()) {
-				unmarshaller.addNamespaceToPackageMapping(mapping.getKey(), mapping.getValue());
-			}
+			this.namespaceToPackageMapping.forEach(unmarshaller::addNamespaceToPackageMapping);
 		}
 		if (this.entityResolver != null) {
 			unmarshaller.setEntityResolver(this.entityResolver);
diff --git a/spring-oxm/src/main/java/org/springframework/oxm/xstream/XStreamMarshaller.java b/spring-oxm/src/main/java/org/springframework/oxm/xstream/XStreamMarshaller.java
@@ -469,15 +469,11 @@ else if (this.converters[i] instanceof SingleValueConverter) {
 		try {
 			if (this.aliases != null) {
 				Map<String, Class<?>> classMap = toClassMap(this.aliases);
-				for (Map.Entry<String, Class<?>> entry : classMap.entrySet()) {
-					xstream.alias(entry.getKey(), entry.getValue());
-				}
+				classMap.forEach(xstream::alias);
 			}
 			if (this.aliasesByType != null) {
 				Map<String, Class<?>> classMap = toClassMap(this.aliasesByType);
-				for (Map.Entry<String, Class<?>> entry : classMap.entrySet()) {
-					xstream.aliasType(entry.getKey(), entry.getValue());
-				}
+				classMap.forEach(xstream::aliasType);
 			}
 			if (this.fieldAliases != null) {
 				for (Map.Entry<String, String> entry : this.fieldAliases.entrySet()) {
@@ -543,20 +539,20 @@ else if (entry.getValue() instanceof List) {
 		}
 
 		if (this.implicitCollections != null) {
-			for (Map.Entry<Class<?>, String> entry : this.implicitCollections.entrySet()) {
-				String[] collectionFields = StringUtils.commaDelimitedListToStringArray(entry.getValue());
+			this.implicitCollections.forEach((key, fields) -> {
+				String[] collectionFields = StringUtils.commaDelimitedListToStringArray(fields);
 				for (String collectionField : collectionFields) {
-					xstream.addImplicitCollection(entry.getKey(), collectionField);
+					xstream.addImplicitCollection(key, collectionField);
 				}
-			}
+			});
 		}
 		if (this.omittedFields != null) {
-			for (Map.Entry<Class<?>, String> entry : this.omittedFields.entrySet()) {
-				String[] fields = StringUtils.commaDelimitedListToStringArray(entry.getValue());
+			this.omittedFields.forEach((key, value) -> {
+				String[] fields = StringUtils.commaDelimitedListToStringArray(value);
 				for (String field : fields) {
-					xstream.omitField(entry.getKey(), field);
+					xstream.omitField(key, field);
 				}
-			}
+			});
 		}
 
 		if (this.annotatedClasses != null) {
diff --git a/spring-test/src/main/java/org/springframework/test/web/servlet/htmlunit/HtmlUnitRequestBuilder.java b/spring-test/src/main/java/org/springframework/test/web/servlet/htmlunit/HtmlUnitRequestBuilder.java
@@ -26,7 +26,6 @@
 import java.util.List;
 import java.util.Locale;
 import java.util.Map;
-import java.util.Map.Entry;
 import java.util.Set;
 import java.util.StringTokenizer;
 import javax.servlet.ServletContext;
@@ -182,11 +181,7 @@ private void parent(MockHttpServletRequest request, @Nullable RequestBuilder par
 
 		// parameter
 		Map<String, String[]> parentParams = parentRequest.getParameterMap();
-		for (Map.Entry<String, String[]> parentParam : parentParams.entrySet()) {
-			String paramName = parentParam.getKey();
-			String[] paramValues = parentParam.getValue();
-			request.addParameter(paramName, paramValues);
-		}
+		parentParams.forEach(request::addParameter);
 
 		// cookie
 		Cookie[] parentCookies = parentRequest.getCookies();
@@ -314,9 +309,7 @@ private String header(String headerName) {
 	}
 
 	private void headers(MockHttpServletRequest request) {
-		for (Entry<String, String> header : this.webRequest.getAdditionalHeaders().entrySet()) {
-			request.addHeader(header.getKey(), header.getValue());
-		}
+		this.webRequest.getAdditionalHeaders().forEach(request::addHeader);
 	}
 
 	private MockHttpSession httpSession(MockHttpServletRequest request, final String sessionid) {
@@ -359,14 +352,13 @@ private void locales(MockHttpServletRequest request) {
 	}
 
 	private void params(MockHttpServletRequest request, UriComponents uriComponents) {
-		for (Entry<String, List<String>> entry : uriComponents.getQueryParams().entrySet()) {
-			String name = entry.getKey();
+		uriComponents.getQueryParams().forEach((name, values) -> {
 			String urlDecodedName = urlDecode(name);
-			for (String value : entry.getValue()) {
+			values.forEach(value -> {
 				value = (value != null ? urlDecode(value) : "");
 				request.addParameter(urlDecodedName, value);
-			}
-		}
+			});
+		});
 		for (NameValuePair param : this.webRequest.getRequestParameters()) {
 			request.addParameter(param.getName(), param.getValue());
 		}
diff --git a/spring-test/src/main/java/org/springframework/test/web/servlet/request/MockHttpServletRequestBuilder.java b/spring-test/src/main/java/org/springframework/test/web/servlet/request/MockHttpServletRequestBuilder.java
diff --git a/spring-tx/src/main/java/org/springframework/transaction/interceptor/MethodMapTransactionAttributeSource.java b/spring-tx/src/main/java/org/springframework/transaction/interceptor/MethodMapTransactionAttributeSource.java
diff --git a/spring-tx/src/main/java/org/springframework/transaction/interceptor/NameMatchTransactionAttributeSource.java b/spring-tx/src/main/java/org/springframework/transaction/interceptor/NameMatchTransactionAttributeSource.java

Original file line number	Diff line number	Diff line change
`@@ -131,13 +131,12 @@ public MapSqlParameterSource addValue(String paramName, Object value, int sqlTyp`
`131`	`131`	`*/`
`132`	`132`	`public MapSqlParameterSource addValues(@Nullable Map<String, ?> values) {`
`133`	`133`	`if (values != null) {`
`134`		`- for (Map.Entry<String, ?> entry : values.entrySet()) {`
`135`		`- this.values.put(entry.getKey(), entry.getValue());`
`136`		`- if (entry.getValue() instanceof SqlParameterValue) {`
`137`		`- SqlParameterValue value = (SqlParameterValue) entry.getValue();`
`138`		`- registerSqlType(entry.getKey(), value.getSqlType());`
	`134`	`+ values.forEach((key, value) -> {`
	`135`	`+ this.values.put(key, value);`
	`136`	`+ if (value instanceof SqlParameterValue) {`
	`137`	`+ registerSqlType(key, ((SqlParameterValue) value).getSqlType());`
`139`	`138`	`}`
`140`		`- }`
	`139`	`+ });`
`141`	`140`	`}`
`142`	`141`	`return this;`
`143`	`142`	`}`
Original file line number	Diff line number	Diff line change
`@@ -288,9 +288,7 @@ protected final void detectHandlerMethods(final Object handler) {`
`288`	`288`	`if (logger.isDebugEnabled()) {`
`289`	`289`	`logger.debug(methods.size() + " message handler methods found on " + userType + ": " + methods);`
`290`	`290`	`}`
`291`		`- for (Map.Entry<Method, T> entry : methods.entrySet()) {`
`292`		`- registerHandlerMethod(handler, entry.getKey(), entry.getValue());`
`293`		`- }`
	`291`	`+ methods.forEach((key, value) -> registerHandlerMethod(handler, key, value));`
`294`	`292`	`}`
`295`	`293`	`}`
`296`	`294`
Original file line number	Diff line number	Diff line change
`@@ -387,11 +387,11 @@ private List<String> getMatchingHeaderNames(String pattern, @Nullable Map<String`
`387`	`387`	`*/`
`388`	`388`	`public void copyHeaders(@Nullable Map<String, ?> headersToCopy) {`
`389`	`389`	`if (headersToCopy != null) {`
`390`		`- for (Map.Entry<String, ?> entry : headersToCopy.entrySet()) {`
`391`		`- if (!isReadOnly(entry.getKey())) {`
`392`		`- setHeader(entry.getKey(), entry.getValue());`
	`390`	`+ headersToCopy.forEach((key, value) -> {`
	`391`	`+ if (!isReadOnly(key)) {`
	`392`	`+ setHeader(key, value);`
`393`	`393`	`}`
`394`		`- }`
	`394`	`+ });`
`395`	`395`	`}`
`396`	`396`	`}`
`397`	`397`
`@@ -401,11 +401,11 @@ public void copyHeaders(@Nullable Map<String, ?> headersToCopy) {`
`401`	`401`	`*/`
`402`	`402`	`public void copyHeadersIfAbsent(@Nullable Map<String, ?> headersToCopy) {`
`403`	`403`	`if (headersToCopy != null) {`
`404`		`- for (Map.Entry<String, ?> entry : headersToCopy.entrySet()) {`
`405`		`- if (!isReadOnly(entry.getKey())) {`
`406`		`- setHeaderIfAbsent(entry.getKey(), entry.getValue());`
	`404`	`+ headersToCopy.forEach((key, value) -> {`
	`405`	`+ if (!isReadOnly(key)) {`
	`406`	`+ setHeaderIfAbsent(key, value);`
`407`	`407`	`}`
`408`		`- }`
	`408`	`+ });`
`409`	`409`	`}`
`410`	`410`	`}`
`411`	`411`