From 1414ab4681fd9f2725da0a0beb6667cf5a741b88 Mon Sep 17 00:00:00 2001
From: Edward Chow <ledwardchow@gmail.com>
Date: Sat, 12 Nov 2022 22:26:08 +1100
Subject: [PATCH] use preconditions instead of try...catch

---
 .../ui/traymenu/AwtTrayMenuController.java    | 19 ++++++++-----------
 1 file changed, 8 insertions(+), 11 deletions(-)

diff --git a/src/main/java/org/cryptomator/ui/traymenu/AwtTrayMenuController.java b/src/main/java/org/cryptomator/ui/traymenu/AwtTrayMenuController.java
index e8fbc570e..4b6a96bca 100644
--- a/src/main/java/org/cryptomator/ui/traymenu/AwtTrayMenuController.java
+++ b/src/main/java/org/cryptomator/ui/traymenu/AwtTrayMenuController.java
@@ -1,5 +1,6 @@
 package org.cryptomator.ui.traymenu;
 
+import dagger.internal.Preconditions;
 import org.apache.commons.lang3.SystemUtils;
 import org.cryptomator.integrations.common.CheckAvailability;
 import org.cryptomator.integrations.common.Priority;
@@ -64,17 +65,13 @@ public class AwtTrayMenuController implements TrayMenuController {
 
 	@Override
 	public void onBeforeOpenMenu(Runnable listener) {
-		try {
-			this.trayIcon.addMouseListener(new MouseAdapter() {
-				@Override
-				public void mouseClicked(MouseEvent e) {
-					listener.run();
-				}
-			});
-		} catch (Exception e) {
-			throw new IllegalStateException("Tray icon not found.", e);
-		}
-
+		Preconditions.checkNotNull(this.trayIcon);
+		this.trayIcon.addMouseListener(new MouseAdapter() {
+			@Override
+			public void mouseClicked(MouseEvent e) {
+				listener.run();
+			}
+		});
 	}
 
 	private void addChildren(Menu menu, List<TrayMenuItem> items) {