Use MethodHandle for faster event bus (#1865)

* Use MethodHandle for faster event bus

* Implement hashCode/equals

* Apply review comments

* Bind to the object directly
Dieser Commit ist enthalten in:
Maddy Miller 2021-08-09 10:06:00 +01:00 committet von dordsor21
Ursprung 57f7c93033
Commit e2b2feb7c5
Es konnte kein GPG-Schlüssel zu dieser Signatur gefunden werden
GPG-Schlüssel-ID: 1E53E88969FFCF0B
3 geänderte Dateien mit 96 neuen und 4 gelöschten Zeilen

Datei anzeigen

@ -22,6 +22,8 @@ package com.sk89q.worldedit.util.eventbus;
import com.google.common.collect.HashMultimap;
import com.google.common.collect.Multimap;
import java.lang.invoke.MethodHandle;
import java.lang.invoke.MethodHandles;
import java.lang.reflect.Method;
/**
@ -59,7 +61,14 @@ class AnnotatedSubscriberFinder implements SubscriberFindingStrategy {
);
}
Class<?> eventType = parameterTypes[0];
EventHandler handler = new MethodEventHandler(annotation.priority(), listener, method);
MethodHandle handle;
try {
handle = MethodHandles.publicLookup().unreflect(method);
} catch (IllegalAccessException e) {
throw new IllegalArgumentException("Method " + method + " failed to unreflect.", e);
}
EventHandler handler = new MethodHandleEventHandler(annotation.priority(), listener, handle, method.getName());
methodsInListener.put(eventType, handler);
}
}

Datei anzeigen

@ -20,6 +20,7 @@
package com.sk89q.worldedit.util.eventbus;
import java.lang.reflect.Method;
import java.util.Objects;
import static com.google.common.base.Preconditions.checkNotNull;
@ -35,7 +36,7 @@ public class MethodEventHandler extends EventHandler {
* Create a new event handler.
*
* @param priority the priority
* @param method the method
* @param method the method
*/
public MethodEventHandler(Priority priority, Object object, Method method) {
super(priority);
@ -72,7 +73,8 @@ public class MethodEventHandler extends EventHandler {
if (!method.equals(that.method)) {
return false;
}
return object != null ? object.equals(that.object) : that.object == null;
return Objects.equals(object, that.object);
}
@Override
@ -81,5 +83,4 @@ public class MethodEventHandler extends EventHandler {
result = 31 * result + method.hashCode();
return result;
}
}

Datei anzeigen

@ -0,0 +1,82 @@
/*
* WorldEdit, a Minecraft world manipulation toolkit
* Copyright (C) sk89q <http://www.sk89q.com>
* Copyright (C) WorldEdit team and contributors
*
* This program is free software: you can redistribute it and/or modify
* it under the terms of the GNU General Public License as published by
* the Free Software Foundation, either version 3 of the License, or
* (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*
* You should have received a copy of the GNU General Public License
* along with this program. If not, see <https://www.gnu.org/licenses/>.
*/
package com.sk89q.worldedit.util.eventbus;
import java.lang.invoke.MethodHandle;
import java.lang.invoke.MethodType;
import java.util.Objects;
public class MethodHandleEventHandler extends EventHandler {
private final MethodHandle methodHandle;
private final String methodName;
private final Object object;
/**
* Create a new event handler that uses MethodHandles to dispatch.
*
* @param priority the priority
* @param object The object to invoke it on
* @param methodHandle The handle to invoke
* @param methodName The name of the method (for equality checks)
*/
protected MethodHandleEventHandler(Priority priority, Object object, MethodHandle methodHandle, String methodName) {
super(priority);
this.object = object;
this.methodHandle = methodHandle.bindTo(object).asType(MethodType.methodType(void.class, Object.class));
this.methodName = methodName;
}
@Override
public void dispatch(Object event) throws Exception {
try {
this.methodHandle.invokeExact(event);
} catch (Exception | Error e) {
throw e;
} catch (Throwable t) {
// If it's not an Exception or Error, throw it wrapped.
throw new Exception(t);
}
}
@Override
public int hashCode() {
return Objects.hash(methodName, object);
}
@Override
public boolean equals(Object o) {
if (this == o) {
return true;
}
if (o == null || getClass() != o.getClass()) {
return false;
}
MethodHandleEventHandler that = (MethodHandleEventHandler) o;
if (!methodName.equals(that.methodName)) {
return false;
}
return Objects.equals(object, that.object);
}
}