Replace Collectors.toList with Stream.toList
Closes gh-29203
This commit is contained in:
committed by
Brian Clozel
parent
9d263668d5
commit
0ccb64fe10
@@ -23,7 +23,6 @@ import java.util.HashMap;
|
||||
import java.util.List;
|
||||
import java.util.Map;
|
||||
import java.util.function.Supplier;
|
||||
import java.util.stream.Collectors;
|
||||
|
||||
import org.springframework.http.MediaType;
|
||||
import org.springframework.lang.Nullable;
|
||||
@@ -88,7 +87,7 @@ public class RequestedContentTypeResolverBuilder {
|
||||
*/
|
||||
public RequestedContentTypeResolver build() {
|
||||
List<RequestedContentTypeResolver> resolvers = (!this.candidates.isEmpty() ?
|
||||
this.candidates.stream().map(Supplier::get).collect(Collectors.toList()) :
|
||||
this.candidates.stream().map(Supplier::get).toList() :
|
||||
Collections.singletonList(new HeaderContentTypeResolver()));
|
||||
|
||||
return exchange -> {
|
||||
|
||||
@@ -20,7 +20,6 @@ import java.util.ArrayList;
|
||||
import java.util.List;
|
||||
import java.util.Objects;
|
||||
import java.util.function.Function;
|
||||
import java.util.stream.Collectors;
|
||||
|
||||
import org.springframework.format.FormatterRegistry;
|
||||
import org.springframework.http.codec.ServerCodecConfigurer;
|
||||
@@ -109,7 +108,7 @@ public class WebFluxConfigurerComposite implements WebFluxConfigurer {
|
||||
|
||||
@Nullable
|
||||
private <T> T createSingleBean(Function<WebFluxConfigurer, T> factory, Class<T> beanType) {
|
||||
List<T> result = this.delegates.stream().map(factory).filter(Objects::nonNull).collect(Collectors.toList());
|
||||
List<T> result = this.delegates.stream().map(factory).filter(Objects::nonNull).toList();
|
||||
if (result.isEmpty()) {
|
||||
return null;
|
||||
}
|
||||
|
||||
@@ -20,7 +20,6 @@ import java.util.List;
|
||||
import java.util.Optional;
|
||||
import java.util.function.Function;
|
||||
import java.util.function.Supplier;
|
||||
import java.util.stream.Collectors;
|
||||
|
||||
import org.reactivestreams.Publisher;
|
||||
import reactor.core.publisher.Flux;
|
||||
@@ -203,7 +202,7 @@ public abstract class BodyExtractors {
|
||||
.orElseGet(() -> {
|
||||
List<MediaType> mediaTypes = context.messageReaders().stream()
|
||||
.flatMap(reader -> reader.getReadableMediaTypes(elementType).stream())
|
||||
.collect(Collectors.toList());
|
||||
.toList();
|
||||
return errorFunction.apply(
|
||||
new UnsupportedMediaTypeException(contentType, mediaTypes, elementType));
|
||||
});
|
||||
|
||||
@@ -17,7 +17,6 @@
|
||||
package org.springframework.web.reactive.function;
|
||||
|
||||
import java.util.List;
|
||||
import java.util.stream.Collectors;
|
||||
|
||||
import org.reactivestreams.Publisher;
|
||||
import reactor.core.publisher.Mono;
|
||||
@@ -386,7 +385,7 @@ public abstract class BodyInserters {
|
||||
|
||||
List<MediaType> supportedMediaTypes = context.messageWriters().stream()
|
||||
.flatMap(reader -> reader.getWritableMediaTypes(bodyType).stream())
|
||||
.collect(Collectors.toList());
|
||||
.toList();
|
||||
|
||||
return new UnsupportedMediaTypeException(mediaType, supportedMediaTypes, bodyType);
|
||||
}
|
||||
|
||||
@@ -30,7 +30,6 @@ import java.util.function.Function;
|
||||
import java.util.function.IntPredicate;
|
||||
import java.util.function.Predicate;
|
||||
import java.util.function.Supplier;
|
||||
import java.util.stream.Collectors;
|
||||
|
||||
import io.micrometer.observation.Observation;
|
||||
import io.micrometer.observation.ObservationRegistry;
|
||||
@@ -122,7 +121,7 @@ class DefaultWebClient implements WebClient {
|
||||
return (CollectionUtils.isEmpty(handlerMap) ? Collections.emptyList() :
|
||||
handlerMap.entrySet().stream()
|
||||
.map(entry -> new DefaultResponseSpec.StatusHandler(entry.getKey(), entry.getValue()))
|
||||
.collect(Collectors.toList()));
|
||||
.toList());
|
||||
};
|
||||
|
||||
|
||||
|
||||
@@ -27,7 +27,6 @@ import java.util.List;
|
||||
import java.util.Locale;
|
||||
import java.util.Map;
|
||||
import java.util.Set;
|
||||
import java.util.stream.Collectors;
|
||||
|
||||
import org.apache.commons.logging.Log;
|
||||
import org.apache.commons.logging.LogFactory;
|
||||
@@ -359,7 +358,7 @@ public class ResourceWebHandler implements WebHandler, InitializingBean {
|
||||
}
|
||||
|
||||
if (isOptimizeLocations()) {
|
||||
result = result.stream().filter(Resource::exists).collect(Collectors.toList());
|
||||
result = result.stream().filter(Resource::exists).toList();
|
||||
}
|
||||
|
||||
this.locationsToUse.clear();
|
||||
|
||||
@@ -300,7 +300,7 @@ public abstract class RequestMappingInfoHandlerMapping extends AbstractHandlerMe
|
||||
return this.partialMatches.stream()
|
||||
.filter(PartialMatch::hasProducesMatch)
|
||||
.map(match -> match.getInfo().getParamsCondition().getExpressions())
|
||||
.collect(Collectors.toList());
|
||||
.toList();
|
||||
}
|
||||
|
||||
/**
|
||||
|
||||
@@ -20,7 +20,6 @@ import java.lang.reflect.Method;
|
||||
import java.util.List;
|
||||
import java.util.concurrent.CompletableFuture;
|
||||
import java.util.concurrent.ExecutionException;
|
||||
import java.util.stream.Collectors;
|
||||
|
||||
import org.springframework.core.DefaultParameterNameDiscoverer;
|
||||
import org.springframework.core.ParameterNameDiscoverer;
|
||||
@@ -69,7 +68,7 @@ public class SyncInvocableHandlerMethod extends HandlerMethod {
|
||||
public List<SyncHandlerMethodArgumentResolver> getResolvers() {
|
||||
return this.delegate.getResolvers().stream()
|
||||
.map(resolver -> (SyncHandlerMethodArgumentResolver) resolver)
|
||||
.collect(Collectors.toList());
|
||||
.toList();
|
||||
}
|
||||
|
||||
/**
|
||||
|
||||
@@ -24,7 +24,6 @@ import java.util.List;
|
||||
import java.util.Map;
|
||||
import java.util.Set;
|
||||
import java.util.concurrent.ConcurrentHashMap;
|
||||
import java.util.stream.Collectors;
|
||||
|
||||
import org.apache.commons.logging.Log;
|
||||
import org.apache.commons.logging.LogFactory;
|
||||
@@ -133,9 +132,9 @@ class ControllerMethodResolver {
|
||||
ConfigurableApplicationContext context) {
|
||||
|
||||
return initResolvers(customResolvers, adapterRegistry, context, false, Collections.emptyList()).stream()
|
||||
.filter(resolver -> resolver instanceof SyncHandlerMethodArgumentResolver)
|
||||
.map(resolver -> (SyncHandlerMethodArgumentResolver) resolver)
|
||||
.collect(Collectors.toList());
|
||||
.filter(SyncHandlerMethodArgumentResolver.class::isInstance)
|
||||
.map(SyncHandlerMethodArgumentResolver.class::cast)
|
||||
.toList();
|
||||
}
|
||||
|
||||
private static List<HandlerMethodArgumentResolver> modelMethodResolvers(
|
||||
|
||||
@@ -21,7 +21,6 @@ import java.util.Arrays;
|
||||
import java.util.List;
|
||||
import java.util.Map;
|
||||
import java.util.Optional;
|
||||
import java.util.stream.Collectors;
|
||||
|
||||
import reactor.core.publisher.Mono;
|
||||
|
||||
@@ -112,7 +111,7 @@ class ModelInitializer {
|
||||
.zip(resultList, objectArray ->
|
||||
Arrays.stream(objectArray)
|
||||
.map(object -> handleResult(((HandlerResult) object), bindingContext))
|
||||
.collect(Collectors.toList()))
|
||||
.toList())
|
||||
.flatMap(Mono::when);
|
||||
}
|
||||
|
||||
|
||||
@@ -23,7 +23,6 @@ import java.util.List;
|
||||
import java.util.Locale;
|
||||
import java.util.Map;
|
||||
import java.util.Optional;
|
||||
import java.util.stream.Collectors;
|
||||
|
||||
import reactor.core.publisher.Flux;
|
||||
import reactor.core.publisher.Mono;
|
||||
@@ -357,7 +356,7 @@ public class ViewResolutionResultHandler extends HandlerResultHandlerSupport imp
|
||||
private List<MediaType> getMediaTypes(List<View> views) {
|
||||
return views.stream()
|
||||
.flatMap(view -> view.getSupportedMediaTypes().stream())
|
||||
.collect(Collectors.toList());
|
||||
.toList();
|
||||
}
|
||||
|
||||
}
|
||||
|
||||
@@ -18,7 +18,6 @@ package org.springframework.web.reactive.result.condition;
|
||||
|
||||
import java.util.Arrays;
|
||||
import java.util.Iterator;
|
||||
import java.util.stream.Collectors;
|
||||
|
||||
import org.junit.jupiter.api.Test;
|
||||
|
||||
@@ -210,7 +209,7 @@ public class PatternsRequestConditionTests {
|
||||
return new PatternsRequestCondition(Arrays
|
||||
.stream(patterns)
|
||||
.map(this.parser::parse)
|
||||
.collect(Collectors.toList()));
|
||||
.toList());
|
||||
}
|
||||
|
||||
}
|
||||
|
||||
@@ -21,7 +21,6 @@ import java.time.Duration;
|
||||
import java.util.Collections;
|
||||
import java.util.List;
|
||||
import java.util.Map;
|
||||
import java.util.stream.Collectors;
|
||||
|
||||
import io.reactivex.rxjava3.core.Single;
|
||||
import org.junit.jupiter.api.BeforeEach;
|
||||
@@ -208,7 +207,7 @@ public class ModelInitializerTests {
|
||||
MethodIntrospector.selectMethods(controller.getClass(), BINDER_METHODS)
|
||||
.stream()
|
||||
.map(method -> new SyncInvocableHandlerMethod(controller, method))
|
||||
.collect(Collectors.toList());
|
||||
.toList();
|
||||
|
||||
WebBindingInitializer bindingInitializer = new ConfigurableWebBindingInitializer();
|
||||
return new InitBinderBindingContext(bindingInitializer, binderMethods);
|
||||
|
||||
Reference in New Issue
Block a user