Re: RFR: 8141211: Convert TraceExceptions to Unified Logging

2015-12-13 Thread David Holmes
Hi Rachel, On 12/12/2015 7:38 AM, Rachel Protacio wrote: Hello! An update and updated webrev. I've rewritten the two portions of code that had used ttyLockers to print in one function-call, since there is no locker equivalent in UL, plus the fact that it will be easier for users to look for one

Re: RFR: 8145153: Convert TraceMonitorInflation to Unified Logging

2015-12-13 Thread David Holmes
Hi Rachel, On 12/12/2015 3:41 AM, Rachel Protacio wrote: Hi David, Thanks for the review! Updated webrev: http://cr.openjdk.java.net/~rprotacio/8145153.01/ You still aren't building the testlibrary in the test. More below ... On 12/10/2015 7:53 PM, David Holmes wrote: Hi Rachel, On 11/12

RFR - JDK-7065236 : To interpret case-insensitive string locale independently

2015-12-13 Thread Harsha Wardhana B
Hello, Please review the fix for bug JDK-7065236. Issue : https://bugs.openjdk.java.net/browse/JDK-7065236 Webrev : http://cr.openjdk.java.net/~jbachorik/sponsorship/7065236/webrev.00 In order to fix the bug, below were t

RFR : JDK-6744127 - NullPointerException at com.sun.tools.jdi.EventRequestManagerImpl.request

2015-12-13 Thread Harsha Wardhana B
Hi All, Please review the fix for bug, Issue : https://bugs.openjdk.java.net/browse/JDK-6744127 Webrev : http://cr.openjdk.java.net/~jbachorik/sponsorship/6744127/webrev.00 The fix synchronizes access to requestList of EventRequestManagerImpl. Thanks Harsha