On 12/11/17 2:31 AM, Shafi Ahmad wrote:
The method hasListeners() is referenced inside MemoryImpl.java and
defined in NotificationEmitterSupport.
This method is not present in NotificationBroadcasterSupport so I
added it to NotificationBroadcasterSupport.
This is a spec change. Is this intentional? I replied in Sept in
reviewing an earlier version [1] that this cannot be backported. If you
intend to make this spec change, it's better to separate this RFE and it
requires CSR.
Jdk10 bug: https://bugs.openjdk.java.net/browse/JDK-8170299
Webrev link: http://cr.openjdk.java.net/~shshahma/8170299/webrev.02/
<http://cr.openjdk.java.net/%7Eshshahma/8170299/webrev.02/>
MemoryImpl.java
189 th.setName("Debugger"); this is not a debugger thread. Maybe rename
it to "MemoryPool notification thread"? Mandy
[1]
http://mail.openjdk.java.net/pipermail/serviceability-dev/2017-September/021836.html
