not use anymore the deprecated method
git-svn-id: https://svn.apache.org/repos/asf/maven/shared/trunk@1590562 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/src/main/java/org/apache/maven/shared/dependency/analyzer/asm/DefaultMethodVisitor.java b/src/main/java/org/apache/maven/shared/dependency/analyzer/asm/DefaultMethodVisitor.java
index 6e91acf..b92065a 100644
--- a/src/main/java/org/apache/maven/shared/dependency/analyzer/asm/DefaultMethodVisitor.java
+++ b/src/main/java/org/apache/maven/shared/dependency/analyzer/asm/DefaultMethodVisitor.java
@@ -91,19 +91,6 @@
*/
}
- // TODO remove
- public void visitMethodInsn( final int opcode, final String owner, final String name, final String desc )
- {
- resultCollector.addName( owner );
- /*
- * NOTE: Merely invoking a method does not impose a direct dependency on its return type nor its parameter
- * types. For example, the code line <code>bean.method(null);</code> only depends on the owner type of the
- * method. A direct dependency is only introduced when the code explicitly references the method's types by
- * means of a variable declaration or a type check/cast. Those cases are handled by other visitor callbacks.
- */
- }
-
- // for java 8 and default method in interfaces
@Override
public void visitMethodInsn( int opcode, String owner, String name, String desc, boolean itf )
{
diff --git a/src/test/java/org/apache/maven/shared/dependency/analyzer/asm/DependencyVisitorTest.java b/src/test/java/org/apache/maven/shared/dependency/analyzer/asm/DependencyVisitorTest.java
index 694b896..3ab84a9 100644
--- a/src/test/java/org/apache/maven/shared/dependency/analyzer/asm/DependencyVisitorTest.java
+++ b/src/test/java/org/apache/maven/shared/dependency/analyzer/asm/DependencyVisitorTest.java
@@ -460,70 +460,70 @@
public void testVisitMethodInsn()
{
- mv.visitMethodInsn( Opcodes.INVOKEVIRTUAL, "a/b/c", "x", "()V" );
+ mv.visitMethodInsn( Opcodes.INVOKEVIRTUAL, "a/b/c", "x", "()V", false );
assertClasses( "a.b.c" );
}
public void testVisitMethodInsnWithPrimitiveArgument()
{
- mv.visitMethodInsn( Opcodes.INVOKEVIRTUAL, "a/b/c", "x", "(I)V" );
+ mv.visitMethodInsn( Opcodes.INVOKEVIRTUAL, "a/b/c", "x", "(I)V", false );
assertClasses( "a.b.c" );
}
public void testVisitMethodInsnWithPrimitiveArrayArgument()
{
- mv.visitMethodInsn( Opcodes.INVOKEVIRTUAL, "a/b/c", "x", "([I)V" );
+ mv.visitMethodInsn( Opcodes.INVOKEVIRTUAL, "a/b/c", "x", "([I)V", false );
assertClasses( "a.b.c" );
}
public void testVisitMethodInsnWithObjectArgument()
{
- mv.visitMethodInsn( Opcodes.INVOKEVIRTUAL, "a/b/c", "x", "(Lx/y/z;)V" );
+ mv.visitMethodInsn( Opcodes.INVOKEVIRTUAL, "a/b/c", "x", "(Lx/y/z;)V", false );
assertClasses( "a.b.c" );
}
public void testVisitMethodInsnWithObjectArguments()
{
- mv.visitMethodInsn( Opcodes.INVOKEVIRTUAL, "a/b/c", "x", "(Lp/q/r;Lx/y/z;)V" );
+ mv.visitMethodInsn( Opcodes.INVOKEVIRTUAL, "a/b/c", "x", "(Lp/q/r;Lx/y/z;)V", false );
assertClasses( "a.b.c" );
}
public void testVisitMethodInsnWithObjectArrayArgument()
{
- mv.visitMethodInsn( Opcodes.INVOKEVIRTUAL, "a/b/c", "x", "([Lx/y/z;)V" );
+ mv.visitMethodInsn( Opcodes.INVOKEVIRTUAL, "a/b/c", "x", "([Lx/y/z;)V", false );
assertClasses( "a.b.c" );
}
public void testVisitMethodInsnWithPrimitiveReturnType()
{
- mv.visitMethodInsn( Opcodes.INVOKEVIRTUAL, "a/b/c", "x", "()I" );
+ mv.visitMethodInsn( Opcodes.INVOKEVIRTUAL, "a/b/c", "x", "()I", false );
assertClasses( "a.b.c" );
}
public void testVisitMethodInsnWithPrimitiveArrayReturnType()
{
- mv.visitMethodInsn( Opcodes.INVOKEVIRTUAL, "a/b/c", "x", "()[I" );
+ mv.visitMethodInsn( Opcodes.INVOKEVIRTUAL, "a/b/c", "x", "()[I", false );
assertClasses( "a.b.c" );
}
public void testVisitMethodInsnWithObjectReturnType()
{
- mv.visitMethodInsn( Opcodes.INVOKEVIRTUAL, "a/b/c", "x", "()Lx/y/z;" );
+ mv.visitMethodInsn( Opcodes.INVOKEVIRTUAL, "a/b/c", "x", "()Lx/y/z;", false );
assertClasses( "a.b.c" );
}
public void testVisitMethodInsnWithObjectArrayReturnType()
{
- mv.visitMethodInsn( Opcodes.INVOKEVIRTUAL, "a/b/c", "x", "()[Lx/y/z;" );
+ mv.visitMethodInsn( Opcodes.INVOKEVIRTUAL, "a/b/c", "x", "()[Lx/y/z;", false );
assertClasses( "a.b.c" );
}