# HG changeset patch
# User darcy
# Date 1363818074 25200
# Node ID 53cc7486d16b5e74edb3f8e48ca5fdae900a0fa1
# Parent  94c16c1ea6eb751cf5be685474d8a5580316ff30
8010427: Refine Method.isDefault implementation
Reviewed-by: acorn, dlsmith

diff -r 94c16c1ea6eb -r 53cc7486d16b 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