On 19 nov 2013, at 20:06, Mandy Chung <mandy.ch...@oracle.com> wrote:
> On 11/15/2013 6:18 AM, Staffan Larsen wrote: >> All, >> >> ReferenceType.visibleMethods() has some problems with diamond shaped >> interface hierarchies. There is more information on the problem in the bug >> report. >> >> The solution here is to maintain a list of interfaces that we have visited >> so that we do not visit interfaces (above the top of the diamond) more than >> once. >> >> This change is targeted for jdk9 so I will hold on to it until the repos >> open up. >> >> webrev: http://cr.openjdk.java.net/~sla/8028430/webrev.00/ >> bug: https://bugs.openjdk.java.net/browse/JDK-8028430 > > This fix looks okay. > > I think these two lines in the tests meant to test different m methods: > > 115 verifyContains(visible, 1, "Two.m(java.lang.String)"); > 116 verifyContains(visible, 1, "Two.m(java.lang.String)”); Cut and paste error I think. One test of this method is enough. I’ve removed one of these lines. > Nits: the test has some inconsistent indentations. Lots of tabs had escaped into the file. Sorry about that. I’ve fixed both of the above an will push with those changes. Thanks for the review, /Staffan > > Mandy
