only wait 5 seconds in total for isSupported() method

infeo · infeo · commit 3ae8fae420e3 · 2023-01-16T19:34:13.000+01:00
diff --git a/src/main/java/org/cryptomator/linux/revealpath/DBusFileMangerRevealPath.java b/src/main/java/org/cryptomator/linux/revealpath/DBusFileMangerRevealPath.java
@@ -59,7 +59,7 @@ public boolean isSupported() {
 			var existsDbusSend = builderExistsDbusSend.start();
 			var existsNautilus = builderExistsNautilus.start();
 			var existsDolphin = builderExistsDolphin.start();
-			if (existsDbusSend.waitFor(5000, TimeUnit.MILLISECONDS) && existsDolphin.waitFor(5000, TimeUnit.MILLISECONDS) && existsDolphin.waitFor(5000, TimeUnit.MILLISECONDS)) {
+			if (existsDbusSend.waitFor(TIMEOUT_THRESHOLD, TimeUnit.MILLISECONDS) && !existsDolphin.isAlive() && !existsNautilus.isAlive()) {
 				return existsDbusSend.exitValue() == 0 && (existsNautilus.exitValue() == 0 | existsDolphin.exitValue() == 0);
 			}
 		} catch (IOException | InterruptedException e) {

Original file line number	Diff line number	Diff line change
`@@ -59,7 +59,7 @@ public boolean isSupported() {`
`59`	`59`	`var existsDbusSend = builderExistsDbusSend.start();`
`60`	`60`	`var existsNautilus = builderExistsNautilus.start();`
`61`	`61`	`var existsDolphin = builderExistsDolphin.start();`
`62`		`- if (existsDbusSend.waitFor(5000, TimeUnit.MILLISECONDS) && existsDolphin.waitFor(5000, TimeUnit.MILLISECONDS) && existsDolphin.waitFor(5000, TimeUnit.MILLISECONDS)) {`
	`62`	`+ if (existsDbusSend.waitFor(TIMEOUT_THRESHOLD, TimeUnit.MILLISECONDS) && !existsDolphin.isAlive() && !existsNautilus.isAlive()) {`
`63`	`63`	`return existsDbusSend.exitValue() == 0 && (existsNautilus.exitValue() == 0 \| existsDolphin.exitValue() == 0);`
`64`	`64`	`}`
`65`	`65`	`} catch (IOException \| InterruptedException e) {`