fixes

JnRouvignac · Nov 29, 2016 · 52183d4 · 52183d4
1 parent d8b2aa2
commit 52183d4
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/...in/src/main/java/org/autorefactor/refactoring/rules/AndroidDrawAllocationRefactoring.java b/...in/src/main/java/org/autorefactor/refactoring/rules/AndroidDrawAllocationRefactoring.java
@@ -79,10 +79,9 @@ public OnDrawTransformer(RefactoringContext ctx, MethodDeclaration onDrawDeclara
             this.onDrawDeclaration = onDrawDeclaration;
         }
 
-        public boolean isMethodBindingSubclassOf(ITypeBinding typeBinding, List<String> superClassStrings) {
+        public boolean isTypeBindingSubclassOf(ITypeBinding typeBinding, List<String> superClassStrings) {
             ITypeBinding superClass = typeBinding;
-            ITypeBinding objectType = ctx.getAST().resolveWellKnownType("java.lang.Object");
-            while (superClass != null && !superClass.equals(objectType)) {
+            while (superClass != null) {
                 if (superClassStrings.contains(superClass.getErasure().getName())) {
                     return true;
                 }
@@ -108,7 +107,7 @@ public boolean visit(VariableDeclarationFragment node) {
                             final ASTBuilder b = this.ctx.getASTBuilder();
                             final Refactorings r = this.ctx.getRefactorings();
                             // Deal with collections
-                            if (isMethodBindingSubclassOf(node.getName().resolveTypeBinding(),
+                            if (isTypeBindingSubclassOf(node.getName().resolveTypeBinding(),
                                     Arrays.asList("AbstractCollection", "Collection", "List", "AbstractMap", "Map"))) {
                                 // It should only works with allocations of
                                 // empty collections.