event-bus-proc/src/main/java/dev/kske/eventbus/proc/EventProcessor.java

@@ -63,10 +63,6 @@ public class EventProcessor extends AbstractProcessor {
 			else
 				pass = true;
 
-			// Warn the user about unused return values
-			if (useParameter && eventHandler.getReturnType().getKind() != TypeKind.VOID)
-					warning(eventHandler, "Unused return value");
-
 			// Abort checking if the handler signature is incorrect
 			if (!pass)
 				continue;
@@ -84,20 +80,14 @@ public class EventProcessor extends AbstractProcessor {
 				}
 			}
 
-			// Detect missing or useless @Polymorphic
-			boolean	polymorphic		= eventHandler.getAnnotation(Polymorphic.class) != null;
-			Element	eventElement	= ((DeclaredType) eventType).asElement();
-
 			// Check for handlers for abstract types that aren't polymorphic
-			if (!polymorphic && (eventElement.getKind() == ElementKind.INTERFACE
-				|| eventElement.getModifiers().contains(Modifier.ABSTRACT)))
+			Element eventElement = ((DeclaredType) eventType).asElement();
+			if (eventHandler.getAnnotation(Polymorphic.class) == null
+				&& (eventElement.getKind() == ElementKind.INTERFACE
+					|| eventElement.getModifiers().contains(Modifier.ABSTRACT))) {
 				warning(eventHandler,
 					"Parameter should be instantiable or handler should use @Polymorphic");
-
-			// Check for handlers for final types that are polymorphic
-			else if (polymorphic && eventElement.getModifiers().contains(Modifier.FINAL))
-				warning(eventHandler,
-					"@Polymorphic should be removed as parameter cannot be subclassed");
+			}
 		}
 	}