Print a better error message when attempting to bridge from void to v…

…oid (#35)
jenkinsci · Nov 17, 2022 · 9040c38 · 9040c38
1 parent 95cee5b
commit 9040c38
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/injector/src/main/java/com/infradna/tool/bridge_method_injector/MethodInjector.java b/injector/src/main/java/com/infradna/tool/bridge_method_injector/MethodInjector.java
@@ -231,6 +231,8 @@ public void inject(ClassVisitor cv) {
                     if (returnType.equals(Type.VOID_TYPE) || returnType.getClassName().equals("java.lang.Void")) {
                         // bridge to void, which means disregard the return value from the original method
                         switch (originalReturnType.getSize()) {
+                        case 0:
+                            throw new IllegalArgumentException("Cannot bridge " + name + " from void to void; did you mean to use a different type?");
                         case 1:
                             mv.visitInsn(POP);
                             break;