This is an automated email from the ASF dual-hosted git repository.

sergeyb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/cxf.git


The following commit(s) were added to refs/heads/master by this push:
     new 5f91cf1  [CXF-7609] Using Optional instead of Collector
5f91cf1 is described below

commit 5f91cf1cbf2a4310a69c156f7de60a07cb728c75
Author: Sergey Beryozkin <[email protected]>
AuthorDate: Tue Jan 16 17:18:05 2018 +0000

    [CXF-7609] Using Optional instead of Collector
---
 .../org/apache/cxf/jaxrs/provider/ServerProviderFactory.java     | 7 +++----
 .../java/org/apache/cxf/jaxrs/client/ClientProviderFactory.java  | 9 +++------
 2 files changed, 6 insertions(+), 10 deletions(-)

diff --git 
a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/ServerProviderFactory.java
 
b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/ServerProviderFactory.java
index 8bb8974..611ab47 100644
--- 
a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/ServerProviderFactory.java
+++ 
b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/ServerProviderFactory.java
@@ -30,7 +30,6 @@ import java.util.List;
 import java.util.Map;
 import java.util.Set;
 import java.util.concurrent.ConcurrentHashMap;
-import java.util.stream.Collectors;
 
 import javax.ws.rs.BeanParam;
 import javax.ws.rs.Priorities;
@@ -188,14 +187,14 @@ public final class ServerProviderFactory extends 
ProviderFactory {
         boolean makeDefaultWaeLeastSpecific =
             MessageUtils.getContextualBoolean(m, 
MAKE_DEFAULT_WAE_LEAST_SPECIFIC, false);
         
-        List<ExceptionMapper<?>> candidates = exceptionMappers.stream()
+        return (ExceptionMapper<T>)exceptionMappers.stream()
                 .filter(em -> handleMapper(em, exceptionType, m, 
ExceptionMapper.class, true))
                 .sorted(new ExceptionProviderInfoComparator(exceptionType,
                                                             
makeDefaultWaeLeastSpecific))
                 .map(ProviderInfo::getProvider)
-                .collect(Collectors.toList());
+                .findFirst()
+                .orElse(null);
         
-        return candidates.isEmpty() ? null : 
(ExceptionMapper<T>)candidates.get(0);
     }
 
 
diff --git 
a/rt/rs/client/src/main/java/org/apache/cxf/jaxrs/client/ClientProviderFactory.java
 
b/rt/rs/client/src/main/java/org/apache/cxf/jaxrs/client/ClientProviderFactory.java
index 9abd320..8807f99 100644
--- 
a/rt/rs/client/src/main/java/org/apache/cxf/jaxrs/client/ClientProviderFactory.java
+++ 
b/rt/rs/client/src/main/java/org/apache/cxf/jaxrs/client/ClientProviderFactory.java
@@ -21,7 +21,6 @@ package org.apache.cxf.jaxrs.client;
 import java.util.ArrayList;
 import java.util.Collections;
 import java.util.List;
-import java.util.stream.Collectors;
 
 import javax.ws.rs.client.ClientRequestFilter;
 import javax.ws.rs.client.ClientResponseFilter;
@@ -103,14 +102,12 @@ public final class ClientProviderFactory extends 
ProviderFactory {
     public <T extends Throwable> ResponseExceptionMapper<T> 
createResponseExceptionMapper(
                                  Message m, Class<?> paramType) {
 
-        List<ResponseExceptionMapper<?>> candidates = 
-            responseExceptionMappers.stream()
+        return (ResponseExceptionMapper<T>)responseExceptionMappers.stream()
                 .filter(em -> handleMapper(em, paramType, m, 
ResponseExceptionMapper.class, true))
                 .map(ProviderInfo::getProvider)
                 .sorted(new ProviderFactory.ClassComparator(paramType))
-                .collect(Collectors.toList());
-             
-        return candidates.isEmpty() ? null : 
(ResponseExceptionMapper<T>)candidates.get(0);
+                .findFirst()
+                .orElse(null);
     }
 
     @Override

-- 
To stop receiving notification emails like this one, please contact
['"[email protected]" <[email protected]>'].

Reply via email to