Properly handle reloads on Spigot.

The code is getting uglier and uglier ...
This commit is contained in:
Kristian S. Stangeland 2013-07-06 08:06:16 +02:00
parent a4f81e5e9f
commit 47a5382709

View File

@ -22,6 +22,7 @@ import java.lang.reflect.Method;
import java.util.Collections; import java.util.Collections;
import java.util.List; import java.util.List;
import java.util.Set; import java.util.Set;
import java.util.concurrent.Callable;
import java.util.concurrent.ConcurrentHashMap; import java.util.concurrent.ConcurrentHashMap;
import java.util.concurrent.atomic.AtomicBoolean; import java.util.concurrent.atomic.AtomicBoolean;
import java.util.concurrent.atomic.AtomicInteger; import java.util.concurrent.atomic.AtomicInteger;
@ -272,14 +273,9 @@ public final class PacketFilterManager implements ProtocolManager, ListenerInvok
delayed.setAsynchronousManager(asyncManager); delayed.setAsynchronousManager(asyncManager);
asyncManager.setManager(delayed); asyncManager.setManager(delayed);
Futures.addCallback(BukkitFutures.nextEvent(library, WorldInitEvent.class), new FutureCallback<WorldInitEvent>() { final Callable<Object> registerSpigot = new Callable<Object>() {
@Override @Override
public void onSuccess(WorldInitEvent event) { public Object call() throws Exception {
// Nevermind
if (delayed.isClosed())
return;
try {
// Now we are probably able to check for Netty // Now we are probably able to check for Netty
InjectedServerConnection inspector = new InjectedServerConnection(reporter, null, server, null); InjectedServerConnection inspector = new InjectedServerConnection(reporter, null, server, null);
Object connection = inspector.getServerConnection(); Object connection = inspector.getServerConnection();
@ -291,9 +287,24 @@ public final class PacketFilterManager implements ProtocolManager, ListenerInvok
delayed.setDelegate(new PacketFilterManager( delayed.setDelegate(new PacketFilterManager(
classLoader, server, library, asyncManager, mcVersion, unhookTask, reporter, useNetty) classLoader, server, library, asyncManager, mcVersion, unhookTask, reporter, useNetty)
); );
// Reference this manager directly // Reference this manager directly
asyncManager.setManager(delayed.getDelegate()); asyncManager.setManager(delayed.getDelegate());
return null;
}
};
// If the server hasn't loaded yet - wait
if (server.getWorlds().size() == 0) {
Futures.addCallback(BukkitFutures.nextEvent(library, WorldInitEvent.class), new FutureCallback<WorldInitEvent>() {
@Override
public void onSuccess(WorldInitEvent event) {
// Nevermind
if (delayed.isClosed())
return;
try {
registerSpigot.call();
} catch (Exception e) { } catch (Exception e) {
onFailure(e); onFailure(e);
} }
@ -305,6 +316,15 @@ public final class PacketFilterManager implements ProtocolManager, ListenerInvok
} }
}); });
} else {
// Do it now
try {
registerSpigot.call();
} catch (Exception e) {
e.printStackTrace();
}
}
// Let plugins use this version instead // Let plugins use this version instead
return delayed; return delayed;
} else { } else {