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

Reply via email to