Don't use WeakReference for Method container

Author: Ghost-chu

src/main/java/org/maxgamer/quickshop/util/reload/ReloadManager.java

@@ -51,7 +51,7 @@ public synchronized void register(@NotNull Reloadable reloadable) {
      */
     public synchronized void register(@NotNull Method reloadMethod) {
         unregister(reloadMethod);
-        this.registry.add(new ReloadableContainer(null, new WeakReference<>(reloadMethod)));
+        this.registry.add(new ReloadableContainer(null, reloadMethod));
     }
 
     /**
@@ -62,7 +62,7 @@ public synchronized void register(@NotNull Method reloadMethod) {
     public synchronized void unregister(@NotNull Method reloadMethod) {
         this.registry.removeIf(reloadableContainer -> {
             if (reloadableContainer.getReloadableMethod() != null) {
-                Method method = reloadableContainer.getReloadableMethod().get();
+                Method method = reloadableContainer.getReloadableMethod();
                 return reloadMethod.equals(method);
             } else {
                 return false;
@@ -91,7 +91,7 @@ public synchronized void unregister(@NotNull Class<Reloadable> clazz) {
                 return clazz.equals(reloadable.getReloadable().getClass());
             }
             if (reloadable.getReloadableMethod() != null) {
-                Method method = reloadable.getReloadableMethod().get();
+                Method method = reloadable.getReloadableMethod();
                 if (method != null)
                     return clazz.equals(method.getDeclaringClass());
             }
@@ -128,7 +128,7 @@ public synchronized Map<ReloadableContainer, ReloadResult> reload(@Nullable Clas
                     }
                 }
                 if (reloadable.getReloadableMethod() != null) {
-                    Method method = reloadable.getReloadableMethod().get();
+                    Method method = reloadable.getReloadableMethod();
                     if (method != null) {
                         if (!clazz.equals(method.getDeclaringClass())) {
                             continue;
@@ -147,7 +147,7 @@ public synchronized Map<ReloadableContainer, ReloadResult> reload(@Nullable Clas
                         reloadResult = new ReloadResult(ReloadStatus.OUTDATED, "Object has been invalid", null);
                     }
                 } else if (reloadable.getReloadableMethod() != null) {
-                    Method method = reloadable.getReloadableMethod().get();
+                    Method method = reloadable.getReloadableMethod();
                     if (method != null) {
                         reloadResult = (ReloadResult) method.invoke(null);
                     } else {

src/main/java/org/maxgamer/quickshop/util/reload/ReloadableContainer.java

@@ -34,7 +34,7 @@ public class ReloadableContainer {
     @Nullable
     private WeakReference<Reloadable> reloadable;
     @Nullable
-    private WeakReference<Method> reloadableMethod;
+    private Method reloadableMethod;
 
     public boolean isObject() {
         return reloadable != null;