RFR: 4987375: (reflect) Class.get{Declared}Method{s} does not return clone() for array type

Joel Borggrén-Franck joel.franck at oracle.com
Fri Sep 6 18:19:09 UTC 2013

Hi Again,

Thanks for the comments.

New webrev: http://cr.openjdk.java.net/~jfranck/4987375/webrev.02/
New specdiff: http://cr.openjdk.java.net/~jfranck/4987375/specdiff.02/

I have added {@code } around class names. Also fixed "array returned" to "returned array" including in two places in get{Declared}Fields() for consistency.

Also rewrote the clone() clause for getMethods().

please review


On Sep 4, 2013, at 3:55 PM, Joel Borggren-Franck <joel.franck at oracle.com> wrote:

> Hi,
> Please review fix for: http://bugs.sun.com/view_bug.do?bug_id=4987375
> Webrev: http://cr.openjdk.java.net/~jfranck/4987375/webrev.01/
> Specdiff: http://cr.openjdk.java.net/~jfranck/4987375/specdiff/java/lang/Class.html
> There are two issues here,
> - First a getInterfaces() call on an array Class instance does return
>  Cloneable and Serializable. This is specified for
>  getGenericInterfaces() but not specified in getInterface(). The fix is
>  to update the spec to match the implementation, which also aligns it
>  with getGenericInterfaces().
> - Also even though JLS states that array types have an implementation of
>  clone() overriding the Object method, it is not included in
>  get{Declared}Method{s}. Again the fix is to note this in the spec.
> Me and Alex have also worked on the structure of the docs trying to
> unify them and have a better flow. Rough outline is:
> <!-- Basic result -->
> <!-- Interesting results -->
> <!-- Empty results -->
> <!-- Order of results -->
> cheers
> /Joel

More information about the core-libs-dev mailing list