You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@groovy.apache.org by jw...@apache.org on 2016/08/23 02:56:14 UTC

[2/5] groovy git commit: findbugs: un-callable method in anonymous class (closes #389)

findbugs: un-callable method in anonymous class (closes #389)


Project: http://git-wip-us.apache.org/repos/asf/groovy/repo
Commit: http://git-wip-us.apache.org/repos/asf/groovy/commit/0a6789d0
Tree: http://git-wip-us.apache.org/repos/asf/groovy/tree/0a6789d0
Diff: http://git-wip-us.apache.org/repos/asf/groovy/diff/0a6789d0

Branch: refs/heads/GROOVY_2_4_X
Commit: 0a6789d06cc6451fcfee174ba638c0494f2827ef
Parents: 2551510
Author: John Wagenleitner <jw...@apache.org>
Authored: Sun Aug 21 16:14:55 2016 -0700
Committer: John Wagenleitner <jw...@apache.org>
Committed: Mon Aug 22 19:17:21 2016 -0700

----------------------------------------------------------------------
 src/main/groovy/lang/MetaClassImpl.java | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/groovy/blob/0a6789d0/src/main/groovy/lang/MetaClassImpl.java
----------------------------------------------------------------------
diff --git a/src/main/groovy/lang/MetaClassImpl.java b/src/main/groovy/lang/MetaClassImpl.java
index 3c49a33..b2c7a0c 100644
--- a/src/main/groovy/lang/MetaClassImpl.java
+++ b/src/main/groovy/lang/MetaClassImpl.java
@@ -497,10 +497,13 @@ public class MetaClassImpl implements MetaClass, MutableMetaClass {
 
         class MOPIter extends MethodIndexAction {
             boolean useThis;
-            public boolean skipClass(CachedClass clazz) {
-                return !useThis && clazz == theCachedClass;
+
+            @Override
+            public boolean skipClass(Class clazz) {
+                return !useThis && clazz == theClass;
             }
 
+            @Override
             public void methodNameAction(Class clazz, MetaMethodIndex.Entry e) {
                 if (useThis) {
                     if (e.methods == null)