Review Request for JDK-8035133: Locale matching: Weight q=0 isn't handled correctly.

2016-08-01 Thread Nishit Jain
Hi, Please review the fix for JDK-8035133 Bug: https://bugs.openjdk.java.net/browse/JDK-8035133 Webrev: http://cr.openjdk.java.net/~nishjain/8035133/webrev.03/ Fix: Changed the filter and lookup mechanism to eliminate the matching tags which are falling in the specified exclusion range i.e. th

Re: Review Request for JDK-7153347: System read/stat/open calls should be hardened to handle EINTR

2016-08-01 Thread Masayoshi Okutsu
+1 Masayoshi On 8/1/2016 11:47 PM, Langer, Christoph wrote: Hi Nishit, this looks good and aligns with other places which were hardened for EINTR. But I'm not a reviewer, so you need to get a higher vote, still :) Best regards Christoph -Original Message- From: i18n-dev [mailto:i1

Re: Review Request for JDK-7153347: System read/stat/open calls should be hardened to handle EINTR

2016-08-01 Thread Langer, Christoph
Hi Nishit, this looks good and aligns with other places which were hardened for EINTR. But I'm not a reviewer, so you need to get a higher vote, still :) Best regards Christoph > -Original Message- > From: i18n-dev [mailto:i18n-dev-boun...@openjdk.java.net] On Behalf Of > Nishit Jain >

Re: RFR: 8161203: ResourceBundle.getBundle performance regression

2016-08-01 Thread Peter Levart
Hi Masayosh, Alan, Thanks for looking at the change. I suppose application containers are already accustomed to invoke ResourceBundle.clearCache(ClassLoader) when undeploying an application so that the corresponding ClassLoader can get GCed right away. But there's a change in the semantics of

Review Request for JDK-7153347: System read/stat/open calls should be hardened to handle EINTR

2016-08-01 Thread Nishit Jain
Hi, Please review the fix for JDK-7153347 Bug: https://bugs.openjdk.java.net/browse/JDK-7153347 Webrev: http://cr.openjdk.java.net/~nishjain/7153347/webrev.01/ Fix: Handled the signal (EINTR) interrupt during the native stat/fstat/lstat/read/open function call using RESTARTABLE macro. Regard