From 4a8dbfbc15b7f2ea4c9eb37aa7bc550f3d9821ba Mon Sep 17 00:00:00 2001
From: Armin Schrenk <armin.schrenk@skymatic.de>
Date: Tue, 20 Jul 2021 12:17:08 +0200
Subject: [PATCH] Code review suggestion

Co-authored-by: Sebastian Stenzel <overheadhunter@users.noreply.github.com>
---
 src/main/java/org/cryptomator/ui/health/CheckExecutor.java | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/main/java/org/cryptomator/ui/health/CheckExecutor.java b/src/main/java/org/cryptomator/ui/health/CheckExecutor.java
index 8417c77b5..5b14bd17c 100644
--- a/src/main/java/org/cryptomator/ui/health/CheckExecutor.java
+++ b/src/main/java/org/cryptomator/ui/health/CheckExecutor.java
@@ -1,5 +1,6 @@
 package org.cryptomator.ui.health;
 
+import com.google.common.collect.Comparators;
 import org.cryptomator.common.vaults.Vault;
 import org.cryptomator.cryptofs.VaultConfig;
 import org.cryptomator.cryptofs.health.api.DiagnosticResult;
@@ -70,9 +71,7 @@ public class CheckExecutor {
 				 var cryptor = CryptorProvider.forScheme(vaultConfig.getCipherCombo()).provide(masterkeyClone, csprng)) {
 				c.getHealthCheck().check(vaultPath, vaultConfig, masterkeyClone, cryptor, diagnosis -> {
 					Platform.runLater(() -> c.getResults().add(Result.create(diagnosis)));
-					if (highestResultSeverity.compareTo(diagnosis.getSeverity()) < 0) {
-						highestResultSeverity = diagnosis.getSeverity();
-					}
+					highestResultSeverity = Comparators.max(highestResultSeverity, diagnosis.getSeverity());
 				});
 			}
 			return null;