8010427: Refine Method.isDefault implementation
authordarcy
Wed, 20 Mar 2013 15:21:14 -0700
changeset 16496 53cc7486d16b
parent 16495 94c16c1ea6eb
child 16497 16465c102a60
8010427: Refine Method.isDefault implementation Reviewed-by: acorn, dlsmith
jdk/src/share/classes/java/lang/reflect/Method.java
--- a/jdk/src/share/classes/java/lang/reflect/Method.java	Wed Mar 20 09:50:07 2013 -0700
+++ b/jdk/src/share/classes/java/lang/reflect/Method.java	Wed Mar 20 15:21:14 2013 -0700
@@ -522,16 +522,19 @@
      * Returns {@code true} if this method is a default
      * method; returns {@code false} otherwise.
      *
-     * A default method is a non-abstract method, that is, a method
-     * with a body, declared in an interface type.
+     * A default method is a public non-abstract instance method, that
+     * is, a non-static method with a body, declared in an interface
+     * type.
      *
      * @return true if and only if this method is a default
      * method as defined by the Java Language Specification.
      * @since 1.8
      */
     public boolean isDefault() {
-        return (getModifiers() & Modifier.ABSTRACT) == 0 &&
-            getDeclaringClass().isInterface();
+        // Default methods are public non-abstract instance methods
+        // declared in an interface.
+        return ((getModifiers() & (Modifier.ABSTRACT | Modifier.PUBLIC | Modifier.STATIC)) ==
+                Modifier.PUBLIC) && getDeclaringClass().isInterface();
     }
 
     // NOTE that there is no synchronization used here. It is correct