RFR: 8009411 : getMethods should not inherit static methods from interfaces

Joe Darcy joe.darcy at oracle.com
Tue Oct 22 07:36:13 UTC 2013


Looks fine Joel; thanks,

-Joe

On 10/21/2013 04:12 AM, Joel Borggrén-Franck wrote:
> Hi All,
>
> http://cr.openjdk.java.net/~jfranck/8009411/webrev.03/
>
> New webrev including the test originally contributed by Peter (I noticed the commit doesn't include Peter in contributed by, I have fixed that in my local copy).
>
> Static methods on implemented interfaces are now ignored in the search for methods in getMethod() , and are filtered out from getMethods() calls.
>
> Tests run:
>
> jdk_lang (including the updated testing with this commit)
> jdk_beans1, jdk_beans2, jdk_beans3
>
> please review
>
> cheers
> /Joel
>
> On 26 sep 2013, at 22:53, Joel Borggrén-Franck <joel.franck at oracle.com> wrote:
>
>> Hi again,
>>
>> New webrev with more tests contributed by Amy Lu.
>>
>> http://cr.openjdk.java.net/~jfranck/8009411/webrev.02/
>>
>> No change to non-test code since webrev.01
>>
>> cheers
>> /Joel
>>
>> On 2013-09-12, Joel Borggrén-Franck wrote:
>>> Hi again,
>>>
>>> New webrev: http://cr.openjdk.java.net/~jfranck/8009411/webrev.01/
>>>
>>> Thanks to Remi and Peter for the quick feedback, I've updated the code
>>> to use for-each as well as fixing getMethod(...).
>>>
>>> Andreas Lundblad also added ~100 testcases for getMethod(), both
>>> positive and negative.
>>>
>>> Please review
>>>
>>> cheers
>>> /Joel
>>>
>>> On 2013-09-09, Joel Borggrén-Franck wrote:
>>>> Hi
>>>>
>>>> Pleaser review fix for 8009411 : getMethods should not inherit static methods from interfaces
>>>>
>>>> The issue is that since we added static methods to interfaces those have erroneously been reflected in getMethods of implementing classes. This fix filters out static interface methods from superinterfaces when adding methods. I have also added a note to the javadoc for both getMembers and getDeclaredMembers pointing this out (though it is implied from JLS). Webrev is based on the clarification to getMethods and friends out for review on this list.
>>>>
>>>> Webrev: http://cr.openjdk.java.net/~jfranck/8009411/webrev.00/
>>>> Bug is here: http://bugs.sun.com/view_bug.do?bug_id=8009411
>>>>
>>>> For oracle reviewers, ccc is approved.
>>>>
>>>> cheers
>>>> /Joel




More information about the core-libs-dev mailing list