RFR 8186961 Class.getFields() does not return fields of previously visited super interfaces/classes.
Paul Sandoz
paul.sandoz at oracle.com
Fri Dec 1 22:52:59 UTC 2017
> On 1 Dec 2017, at 12:16, mandy chung <mandy.chung at oracle.com> wrote:
>
>
>
> On 11/30/17 12:17 PM, Paul Sandoz wrote:
>> Here is the updated webrev:
>>
>>
>> http://cr.openjdk.java.net/~psandoz/jdk10/JDK-8186961-iface-static-fields-get/webrev/
>>
>>
>> I opted for the simple solution using a LinkedHashSet.
>>
>
> This fix looks good except a typo in the test:
>
> 94 "Class %s does not have extracly one field: %d", c.getName(), nfs));
>
> s/extracly/exactly/
>
Fixed.
> I am wondering if these @run should run with both othervm and agentvm mode since
> it currently depends on the jtreg command-line
> (I believe our test target uses
> agentvm as the default).
>
It does.
* @run main/othervm StaticFieldsOnInterface C
* @run main/othervm StaticFieldsOnInterface D
* @run main/othervm StaticFieldsOnInterface Y
This ok?
>> It’s possible to heavily optimize (avoiding the production of a linked hash set until required [*]) but the resulting code is harder to understand.
>>
>
> I was tempted to come up with optimization too when first reading the patch but I am with you. I like the resulting code simple and clear. The difference is an array list vs linked hash set which we should wait until this is really a performance issue. FWIW getMethods implementation also creates a linked hash set if not cached.
>
Ok, thanks,
Paul.
More information about the core-libs-dev
mailing list