Declare that event listeners do not have to be public

Author: zml2008

src/ap/java/org/spongepowered/plugin/processor/ListenerProcessor.java

@@ -25,6 +25,7 @@
 package org.spongepowered.plugin.processor;
 
 import static javax.tools.Diagnostic.Kind.ERROR;
+import static javax.tools.Diagnostic.Kind.WARNING;
 
 import org.spongepowered.api.event.Event;
 import org.spongepowered.api.event.Listener;
@@ -50,7 +51,7 @@
 import javax.tools.Diagnostic;
 
 @SupportedAnnotationTypes(ListenerProcessor.LISTENER_ANNOTATION_CLASS)
-@SupportedSourceVersion(SourceVersion.RELEASE_8)
+@SupportedSourceVersion(SourceVersion.RELEASE_16)
 public class ListenerProcessor extends AbstractProcessor {
 
     static final String LISTENER_ANNOTATION_CLASS = "org.spongepowered.api.event.Listener";
@@ -68,19 +69,16 @@ public boolean process(final Set<? extends TypeElement> annotations, final Round
 
                 final Messager msg = this.processingEnv.getMessager();
                 if (method.getModifiers().contains(Modifier.STATIC)) {
-                    msg.printMessage(Diagnostic.Kind.ERROR, "method must not be static", method);
-                }
-                if (!method.getModifiers().contains(Modifier.PUBLIC)) {
-                    msg.printMessage(Diagnostic.Kind.ERROR, "method must be public", method);
+                    msg.printMessage(Diagnostic.Kind.ERROR, "Event listener method must not be static", method);
                 }
                 if (method.getModifiers().contains(Modifier.ABSTRACT)) {
-                    msg.printMessage(Diagnostic.Kind.ERROR, "method must not be abstract", method);
+                    msg.printMessage(Diagnostic.Kind.ERROR, "Event listener method must not be abstract", method);
                 }
                 if (method.getEnclosingElement().getKind().isInterface()) {
                     msg.printMessage(Diagnostic.Kind.ERROR, "interfaces cannot declare listeners", method);
                 }
                 if (method.getReturnType().getKind() != TypeKind.VOID) {
-                    msg.printMessage(Diagnostic.Kind.ERROR, "method must return void", method);
+                    msg.printMessage(Diagnostic.Kind.ERROR, "Event listener method must return void", method);
                 }
                 final List<? extends VariableElement> parameters = method.getParameters();
                 if (parameters.isEmpty() || !this.isTypeSubclass(parameters.get(0), ListenerProcessor.EVENT_CLASS)) {

src/main/java/org/spongepowered/api/event/EventManager.java

@@ -36,13 +36,11 @@ public interface EventManager {
      * Registers {@link Event} methods annotated with @{@link Listener} in the
      * specified object.
      *
-     * <p>Only methods that are public will be registered and the class must be
-     * public as well.</p>
+     * <p>This will not include any methods declared in supertypes, but will
+     * test for private and package-private listener methods.</p>
      *
-     * @param plugin The plugin instance
+     * @param plugin The plugin container
      * @param obj The object
-     * @throws IllegalArgumentException Thrown if {@code plugin} is not a plugin
-     *         instance
      */
     void registerListeners(PluginContainer plugin, Object obj);
 

src/main/java/org/spongepowered/api/event/Listener.java

@@ -32,9 +32,6 @@
 
 /**
  * Used to annotate a method as an {@link EventListener}.
- *
- * <p>The method being targeted must be public and must be in a class that is
- * also public.</p>
  */
 @Retention(RUNTIME)
 @Target(METHOD)