Skip to content

Commit 38ab319

Browse files
committed
/issues/57 Moved sorting to default service loader.
1 parent ad4110a commit 38ab319

File tree

2 files changed

+2
-5
lines changed

2 files changed

+2
-5
lines changed

src/main/java/javax/money/spi/Bootstrap.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,6 @@ public static <T> T getService(Class<T> serviceType) {
113113
List<T> services = getServiceProvider().getServices(serviceType);
114114
return services
115115
.stream()
116-
.sorted(Comparator.comparing(o -> o.getClass().getSimpleName()))
117116
.findFirst()
118117
.orElse(null);
119118
}

src/main/java/javax/money/spi/DefaultServiceProvider.java

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,10 +8,7 @@
88
*/
99
package javax.money.spi;
1010

11-
import java.util.ArrayList;
12-
import java.util.Collections;
13-
import java.util.List;
14-
import java.util.ServiceLoader;
11+
import java.util.*;
1512
import java.util.concurrent.ConcurrentHashMap;
1613
import java.util.logging.Level;
1714
import java.util.logging.Logger;
@@ -65,6 +62,7 @@ private <T> List<T> loadServices(final Class<T> serviceType) {
6562
for (T t : ServiceLoader.load(serviceType)) {
6663
services.add(t);
6764
}
65+
services.sort(Comparator.comparing(o -> o.getClass().getSimpleName()));
6866
@SuppressWarnings("unchecked")
6967
final List<T> previousServices = (List<T>) servicesLoaded.putIfAbsent(serviceType, (List<Object>) services);
7068
return Collections.unmodifiableList(previousServices != null ? previousServices : services);

0 commit comments

Comments
 (0)