Re: Unexpected BindException in Endpoint.publish

2016-08-01 Thread KUBOTA Yuji
Hi all, Could you let me know when this fix will be backport to JDK 8? We need this backport to migrate our system from JDK 7 to JDK 8. Thanks, Yuji 2016-02-10 10:17 GMT+09:00 KUBOTA Yuji : > Hi Miroslav, > > Thank you for your sponsor! :

Re: [PATCH] JDK-8155102: Process.toString could include pid, isAlive, exitStatus

2016-08-01 Thread Andrey Dyachkov
Roger, I have changed impl according to your comments, thank you! Regarding fc-extension, I am not able to add comments in the bug tracker I think you can do it, correct? Also I am already in OCA list. diff --git a/src/java.base/unix/classes/java/lang/ProcessImpl.java

Re: RFR(XS): 8162670: make of jtreg_tests fails if no tests are run, causing jprt test runs to also fail

2016-08-01 Thread Chris Plummer
On 8/1/16 7:25 PM, David Holmes wrote: On 2/08/2016 12:11 PM, Chris Plummer wrote: On 8/1/16 5:58 PM, David Holmes wrote: Hi Chris, On 2/08/2016 8:46 AM, Chris Plummer wrote: Hello, Please review this simple change: https://bugs.openjdk.java.net/browse/JDK-8162670

Re: RFR(XS): 8162670: make of jtreg_tests fails if no tests are run, causing jprt test runs to also fail

2016-08-01 Thread David Holmes
On 2/08/2016 12:11 PM, Chris Plummer wrote: On 8/1/16 5:58 PM, David Holmes wrote: Hi Chris, On 2/08/2016 8:46 AM, Chris Plummer wrote: Hello, Please review this simple change: https://bugs.openjdk.java.net/browse/JDK-8162670 http://cr.openjdk.java.net/~cjplummer/8162670/webrev-00/ You've

Re: RFR(XS): 8162670: make of jtreg_tests fails if no tests are run, causing jprt test runs to also fail

2016-08-01 Thread Chris Plummer
On 8/1/16 5:58 PM, David Holmes wrote: Hi Chris, On 2/08/2016 8:46 AM, Chris Plummer wrote: Hello, Please review this simple change: https://bugs.openjdk.java.net/browse/JDK-8162670 http://cr.openjdk.java.net/~cjplummer/8162670/webrev-00/ You've split a compound expression with your code:

Re: RFR(XS): 8162670: make of jtreg_tests fails if no tests are run, causing jprt test runs to also fail

2016-08-01 Thread David Holmes
Hi Chris, On 2/08/2016 8:46 AM, Chris Plummer wrote: Hello, Please review this simple change: https://bugs.openjdk.java.net/browse/JDK-8162670 http://cr.openjdk.java.net/~cjplummer/8162670/webrev-00/ You've split a compound expression with your code: 227 jtregExitCode=$$? && \ 228 if

RFR(XS): 8162670: make of jtreg_tests fails if no tests are run, causing jprt test runs to also fail

2016-08-01 Thread Chris Plummer
Hello, Please review this simple change: https://bugs.openjdk.java.net/browse/JDK-8162670 http://cr.openjdk.java.net/~cjplummer/8162670/webrev-00/ Note the copyright dates haven't been updated in this webrev, but I did update them locally after noticing that. Tested with jprt test case

JDK 9 RFR of JDK-8162817: Annotation toString output not reusable for source input

2016-08-01 Thread joe darcy
As pointed out by recent core-libs-dev traffic ([1] [2]), the toString output for annotation objects cannot be directly reused as source input if the annotation values are any of: * Class values of array classes (uses JVM name) * Arrays (output uses brackets ("[]") rather than braces {"{}")

[9] RFR: 8162797: Code clean-up in IncludeLocalesPlugin

2016-08-01 Thread Naoto Sato
Hello, Please review this small change for the bug: https://bugs.openjdk.java.net/browse/JDK-8162797 The fix is located at: http://cr.openjdk.java.net/~naoto/8162797/webrev.00/ Naoto

Re: RFR: 8156500: deadlock provoked by new stress test com/sun/jdi/OomDebugTest.java

2016-08-01 Thread Kim Barrett
This updated webrev addresses concerns about the performance of the VM API used by the reference processor thread in the original webrev. New webrevs: full: http://cr.openjdk.java.net/~kbarrett/8156500/jdk.03/ http://cr.openjdk.java.net/~kbarrett/8156500/hotspot.03/ incr:

Re: RFR: 8156500: deadlock provoked by new stress test com/sun/jdi/OomDebugTest.java

2016-08-01 Thread Kim Barrett
This updated webrev addresses the concerns around initialization order for Throwable and OutOfMemoryError. It doesn't address the VM API used by the reference processor thread; that will be in a later webrev. New webrevs: full: http://cr.openjdk.java.net/~kbarrett/8156500/hotspot.02/ incr:

Re: Why is there no Math.ceilDiv?

2016-08-01 Thread Brian Burkhalter
Hello Philipp, On Jul 31, 2016, at 12:45 PM, Philipp Nowak wrote: > I hope this is the right place to ask this question. If it is not, I would > kindly ask you to direct me to the correct place. The is the correct forum. > I was wondering why there is no Math.ceilDiv()

RE: RFR: 8162439: Runtime.Version.parse needs fast-path for major versions

2016-08-01 Thread Iris Clark
Hi, Claes. > http://cr.openjdk.java.net/~redestad/8162439/webrev.02/ This looks good. Thanks, Iris

Why is there no Math.ceilDiv?

2016-08-01 Thread Philipp Nowak
Hello, I hope this is the right place to ask this question. If it is not, I would kindly ask you to direct me to the correct place. I was wondering why there is no Math.ceilDiv() method, when a floorDiv() method got added in Java 8 through JDK-8023217 [1]. Is there any reason why such method

Re: [PATCH] JDK-8155102: Process.toString could include pid, isAlive, exitStatus

2016-08-01 Thread Roger Riggs
Hi Andrey, Sorry to be slow in getting back to this. Thanks for the update; did you notice that Windows has a separate implementation of ProcessImpl and would also need a similar, though not identical toString() implementation?

Re: RFR 8157135, Fix module dependencies javax/* EE tests

2016-08-01 Thread Alan Bateman
On 31/07/2016 23:48, Felix Yang wrote: Hi all, please review the patch for some tests, which explicitly declare module dependencies to EE modules. Bug: https://bugs.openjdk.java.net/browse/JDK-8157135 Webrev: http://cr.openjdk.java.net/~xiaofeya/8157135/webrev.00/ This is a partial fix

RFR 8157135, Fix module dependencies javax/* EE tests

2016-08-01 Thread Felix Yang
Hi all, please review the patch for some tests, which explicitly declare module dependencies to EE modules. Bug: https://bugs.openjdk.java.net/browse/JDK-8157135 Webrev: http://cr.openjdk.java.net/~xiaofeya/8157135/webrev.00/ This is a partial fix left by