[PATCH] 8188240: Reflection Proxy should skip static methods

Sundararajan Athijegannathan sundararajan.athijegannathan at oracle.com
Wed Mar 14 03:32:19 UTC 2018


Looks good

-Sundar

On 14/03/18, 5:49 AM, David Lloyd wrote:
> On Tue, Mar 13, 2018 at 7:16 PM, David Lloyd<david.lloyd at redhat.com>  wrote:
>> On Tue, Mar 13, 2018 at 6:31 PM, mandy chung<mandy.chung at oracle.com>  wrote:
>>> I prefer to keep the original test case i.e. create a proxy class from
>>> Runnable and Observer.   Instead add a new test case to create a proxy class
>>> with ClashWithRunnable, Runnable and Observer and verify that it does not
>>> include static methods.  I would add another static method
>>> ClashWithRunnable::foo (no name clash) and verify
>>> proxyClass::getDeclaredMethods does not contain foo.
>> OK, done.  It's a little bigger now so I'm attaching it.
> Apologies, my IDE put it in the wrong directory.  Here's a better one.
> I ran it by hand as jtreg and I are not presently on speaking terms.


More information about the core-libs-dev mailing list