+1
Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com Sent from my iPad > On Feb 26, 2015, at 8:28 PM, joe darcy <joe.da...@oracle.com> wrote: > > Hello, > > Please review the small changes to address > > JDK-8073952: Spec of j.l.r.Method.toString/toGenericString need to be > clarified > http://cr.openjdk.java.net/~darcy/8073952.0/ > > Full patch below. > > In essence, the patch adds references to the JLS sections which talk about > interfaces and default methods. > > Thanks, > > -Joe > > --- old/src/java.base/share/classes/java/lang/reflect/Constructor.java > 2015-02-26 17:14:49.955169685 -0800 > +++ new/src/java.base/share/classes/java/lang/reflect/Constructor.java > 2015-02-26 17:14:49.799091677 -0800 > @@ -296,7 +296,8 @@ > * constructor has default (package) access. > * > * @return a string describing this {@code Constructor} > - * @jls 8.8.3. Constructor Modifiers > + * @jls 8.8.3 Constructor Modifiers > + * @jls 8.9.2 Enum Body Declarations > */ > public String toString() { > return sharedToString(Modifier.constructorModifiers(), > @@ -342,7 +343,8 @@ > * include type parameters > * > * @since 1.5 > - * @jls 8.8.3. Constructor Modifiers > + * @jls 8.8.3 Constructor Modifiers > + * @jls 8.9.2 Enum Body Declarations > */ > @Override > public String toGenericString() { > --- old/src/java.base/share/classes/java/lang/reflect/Method.java 2015-02-26 > 17:14:50.403393705 -0800 > +++ new/src/java.base/share/classes/java/lang/reflect/Method.java 2015-02-26 > 17:14:50.227305697 -0800 > @@ -356,6 +356,8 @@ > * @return a string describing this {@code Method} > * > * @jls 8.4.3 Method Modifiers > + * @jls 9.4 Method Declarations > + * @jls 9.6.1 Annotation Type Elements > */ > public String toString() { > return sharedToString(Modifier.methodModifiers(), > @@ -409,6 +411,8 @@ > * @since 1.5 > * > * @jls 8.4.3 Method Modifiers > + * @jls 9.4 Method Declarations > + * @jls 9.6.1 Annotation Type Elements > */ > @Override > public String toGenericString() { >
