Hi Anton, Some comments/questions: - Some copyright dates need updating - Line 1112 of JavaComponentAccessibility: does the release of jaccessible cause a release of jparent? - Line 7155 of Component.java: is that the only place where this means is needed?
Pete On 12/22/15 8:10 AM, Anton Tarasov wrote: > Hi guys! > > Could you please review the problem I’ve filed and the suggested fix? > > bug: JDK-8145984 > <https://bugs.openjdk.java.net/browse/JDK-8145984> > sun.lwawt.macosx.CAccessible > leaks > webrev: http://cr.openjdk.java.net/~ant/JDK-8145984/jdk9/webrev.0 > <http://cr.openjdk.java.net/%7Eant/JDK-8145984/jdk9/webrev.0> > > (This is to be addressed in 8u/9. The webrev for 8u is in JIRA, it’s > identical except the paths.) > > Please, find the details in JIRA. > > Thanks! > Anton.