By implementing the full interface instead of extending the EmptyVisitor, future additions to the interface can be discovered at runtime instead of being ignored.

Author: Kiskae

src/main/java/nl/rug/jbi/jsm/bcel/ClassVisitor.java

@@ -21,7 +21,7 @@
  * @see nl.rug.jbi.jsm.core.pipeline.Pipeline#registerNewBaseData(Class)
  * @since 2014-05-28
  */
-public class ClassVisitor extends EmptyVisitor {
+public class ClassVisitor implements Visitor {
     public final static ImmutableSet<Class> DEFAULT_CLASSES = ImmutableSet.<Class>builder()
             .add(JavaClassDefinition.class)
             .add(MethodDefinition.class)

src/main/java/nl/rug/jbi/jsm/bcel/MethodVisitor.java

@@ -15,7 +15,7 @@
  * @author David van Leusen
  * @since 2014-05-28
  */
-public class MethodVisitor extends EmptyVisitor {
+public class MethodVisitor implements Visitor {
     private final static Logger logger = LogManager.getLogger(MethodVisitor.class);
 
     private final MethodGen mg;