From e9c21bd46556e77bb5ac064c670e88f542f8b3de Mon Sep 17 00:00:00 2001 From: Alexander Kriegisch Date: Fri, 12 Apr 2024 13:27:47 +0200 Subject: [PATCH] SignaturePattern.TypePatternVisitor: remove redundant visit method Method visit(WildAnnotationTypePattern, Object) used to descend into node.getTypePattern().accept(this, data), which since commit 6585b9ef46 is unnecessary, because WildAnnotationTypePattern::traverse already traverses its type pattern. Signed-off-by: Alexander Kriegisch --- .../java/org/aspectj/weaver/patterns/SignaturePattern.java | 6 ------ 1 file changed, 6 deletions(-) diff --git a/org.aspectj.matcher/src/main/java/org/aspectj/weaver/patterns/SignaturePattern.java b/org.aspectj.matcher/src/main/java/org/aspectj/weaver/patterns/SignaturePattern.java index 0fcbebe49..e5d2c7e26 100644 --- a/org.aspectj.matcher/src/main/java/org/aspectj/weaver/patterns/SignaturePattern.java +++ b/org.aspectj.matcher/src/main/java/org/aspectj/weaver/patterns/SignaturePattern.java @@ -199,12 +199,6 @@ public TypePatternVisitor(IScope scope, boolean targetsOtherThanTypeAllowed, boo this.parameterTargettingAnnotationsAllowed = parameterTargettingAnnotationsAllowed; } - @Override - public Object visit(WildAnnotationTypePattern node, Object data) { - node.getTypePattern().accept(this, data); - return node; - } - /** * Do the ExactAnnotationTypePatterns have the incorrect target? */