RFR: 8062773: Clarifications for Class specification

joe darcy joe.darcy at oracle.com
Wed Nov 5 05:26:48 UTC 2014

Hi Martin,

This description in getGenericInterfaces

  876      * <p> If this object represents a class, the return value is an
  877      * array containing objects representing all interfaces
  878      * implemented by the class. The order of the interface objects in

could also be direct-ized.

In getGenericInterfaces, the new text

  899      * <p>If this {@code Class} object represents an array type, the
  900      * interfaces {@code Cloneable} and {@code 
java.io.Serializable} are
  901      * returned in that order.

is redundant with the existing text

  881      * represented by this object.  In the case of an array class, the
  882      * interfaces {@code Cloneable} and {@code Serializable} are
  883      * returned in that order.

Exactly one copy of the Cloneable & Serializable assertion should be 
removed :-)

With this change to getFields,

1540      * returns the public fields of the class and of all its 
superclasses and
1541      * superinterfaces.

the bug will need a ccc request since the specification is changing 
(even if it is changing to match long-standing behavior).



On 11/4/2014 9:08 AM, Martin Buchholz wrote:
> [+core-libs-dev]
> I'm sorry I keep forgetting to add the mailing list to my review
> requests.  I blame google's code review mail tool, which cc's the
> right mailing lists automagically.
> Webrev updated to add a few more "direct"s, as suggested by Paul.
> https://bugs.openjdk.java.net/browse/JDK-8062773
> http://cr.openjdk.java.net/~martin/webrevs/openjdk9/Class-spec/
> On Tue, Nov 4, 2014 at 3:02 AM, Paul Sandoz <paul.sandoz at oracle.com> wrote:
>> On Nov 3, 2014, at 11:35 PM, Martin Buchholz <martinrb at google.com> wrote:
>> Hi Joe, Paul,
>> I'd like you to do a code review.
>> https://bugs.openjdk.java.net/browse/JDK-8062773
>> http://cr.openjdk.java.net/~martin/webrevs/openjdk9/Class-spec/
>> Looks ok to me.
>>   730      * @return the superclass of the class represented by this object.
>> You could tweak that to say "the direct superclass..."
>> Paul.

More information about the core-libs-dev mailing list