[PATCH] 8188240: Reflection Proxy should skip static methods
peter.levart at gmail.com
Thu Mar 15 07:56:31 UTC 2018
In test, the following comment:
26 * @summary This is a test to ensure that proxies do not inherit
I think the word "inherit" is not correct here. Interface static methods
can not be inherited. VM already ensures that. Perhaps the comment
"This is a test to ensure that proxies do not try to override
interface static methods."
Nothing can actually override an interface static method even if it
tries to (in the sense class static methods may be overridden). So I
think this test checks that proxy class doesn't try to do this, right?
On 03/14/2018 11:04 PM, Aleksey Shipilev wrote:
> On 03/14/2018 10:44 PM, mandy chung wrote:
>> David - I think the test fails even in your first version.
>> It should use ProxyClashTest.class.getClassLoader() to define the proxy class as the test is running
>> in agent vm mode.
> Right. This passes local testing:
> ...and I am going to redo jdk/submit.
More information about the core-libs-dev