From 3202a1086deac6eb2b9429b73bea8744d33df91a Mon Sep 17 00:00:00 2001 From: jvoisin Date: Sat, 8 Jun 2019 15:50:15 +0200 Subject: [PATCH] Minor simplification `.sorted(Comparator.reverseOrder()).findFirst()` can be replaced with `max(Comparator.naturalOrder()` --- .../main/java/org/airsonic/player/service/VersionService.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/airsonic-main/src/main/java/org/airsonic/player/service/VersionService.java b/airsonic-main/src/main/java/org/airsonic/player/service/VersionService.java index 68aa60be..395b0881 100644 --- a/airsonic-main/src/main/java/org/airsonic/player/service/VersionService.java +++ b/airsonic-main/src/main/java/org/airsonic/player/service/VersionService.java @@ -257,7 +257,7 @@ public class VersionService { Predicate finalVersionPredicate = version -> !version.isPreview(); - Optional betaV = unsortedTags.stream().map(convertToVersion).sorted(Comparator.reverseOrder()).findFirst(); + Optional betaV = unsortedTags.stream().map(convertToVersion).max(Comparator.naturalOrder()); Optional finalV = unsortedTags.stream().map(convertToVersion).sorted(Comparator.reverseOrder()).filter(finalVersionPredicate).findFirst(); LOG.debug("Got {} for beta version", betaV);