From d30758a2a87fec9e228aef2227caf4eb7939527f Mon Sep 17 00:00:00 2001
From: yushijinhun <yushijinhun@gmail.com>
Date: Sat, 21 Jul 2018 09:57:47 +0800
Subject: [PATCH] Throw NPE when selected server is null

---
 HMCL/src/main/java/org/jackhuang/hmcl/ui/AddAccountPane.java | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/HMCL/src/main/java/org/jackhuang/hmcl/ui/AddAccountPane.java b/HMCL/src/main/java/org/jackhuang/hmcl/ui/AddAccountPane.java
index 4d03f2d7f..1b66d1f4c 100644
--- a/HMCL/src/main/java/org/jackhuang/hmcl/ui/AddAccountPane.java
+++ b/HMCL/src/main/java/org/jackhuang/hmcl/ui/AddAccountPane.java
@@ -50,10 +50,10 @@ import org.jackhuang.hmcl.util.Constants;
 import org.jackhuang.hmcl.util.Logging;
 
 import java.util.List;
-import java.util.Optional;
 import java.util.concurrent.CountDownLatch;
 import java.util.logging.Level;
 
+import static java.util.Objects.requireNonNull;
 import static org.jackhuang.hmcl.setting.ConfigHolder.CONFIG;
 import static org.jackhuang.hmcl.ui.FXUtils.jfxListCellFactory;
 import static org.jackhuang.hmcl.ui.FXUtils.onInvalidating;
@@ -125,8 +125,7 @@ public class AddAccountPane extends StackPane {
     private Object getAuthAdditionalData() {
         AccountFactory<?> factory = cboType.getSelectionModel().getSelectedItem();
         if (factory == Accounts.FACTORY_AUTHLIB_INJECTOR) {
-            // throw an exception if none is selected
-            return Optional.ofNullable(cboServers.getSelectionModel().getSelectedItem()).get();
+            return requireNonNull(cboServers.getSelectionModel().getSelectedItem(), "selected server cannot be null");
         }
         return null;
     }