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

Mandy Chung mandy.chung at oracle.com
Thu Sep 5 19:34:11 UTC 2013


On 9/4/13 6:55 AM, Joel Borggren-Franck 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
>
> ...
>
> Me and Alex have also worked on the structure of the docs trying to
> unify them and have a better flow. Rough outline is:
>
This spec clarification and the uniform structure looks good.

Might be nit-picking - you can consider to clean up either with this fix 
or in any future of your revision to the javadoc of java.lang.Class.

Some in the javadoc refers "this Class object" as "this object" and 
either way will work  Type names in javadoc are with {@code ....} tag as 
you have done in other places. s/this Class object/this {@code Class} 
object/ - there are several occurrences that should be fixed (in line 
824, 1536, 1540, 1544, 1548, 1551, 1709, ... more).  Similarly 
s/Method/{@code Method}/ in line 1538, 1542

L1554, 1892 - s/array returned/returned array/
I think that what you try to make consistent.

Mandy




More information about the core-libs-dev mailing list