Re: Process trees and termination

2014-06-26 Thread Peter Levart
On 06/26/2014 12:05 AM, David M. Lloyd wrote: On 06/25/2014 04:52 PM, roger riggs wrote: Hi, The next step for JEP 102, Process API Updates is to provide an API to enumerate the hierarchy of processes and to be able to terminate a process tree. This draft javadoc

Re: Long valueOf instead of new Long

2014-06-26 Thread Chris Hegarty
Otavio, I skimmed over the patches and they look ok to me. I think they would be suitable for inclusion in jdk9/dev. -Chris. P.S. I do not currently have time to sponsor this, but I cc’ed Pavel who may be able to help get his in. On 14 Jun 2014, at 14:46, Otávio Gonçalves de Santana

Re: Long valueOf instead of new Long

2014-06-26 Thread Otávio Gonçalves de Santana
Thank you Chris. I don't know if is better, but I did with all wrapper in this path. http://mail.openjdk.java.net/pipermail/core-libs-dev/2014-June/027285.html But if will better, I can split each wrapper as path. On Thu, Jun 26, 2014 at 6:58 AM, Chris Hegarty chris.hega...@oracle.com wrote:

Re: JDK9 project: XML/JAXP Approachability / Ease of Use

2014-06-26 Thread Michael Kay
Here are some quick thoughts about the state of XML support in the JDK: 1. XML Parser. The version of Xerces in the JDK has long been buggy, and no-one has been fixing the bugs. It needs to be replaced with a more recent version of Apache Xerces if that hasn't already been done. 2. DOM. From a

Re: Long valueOf instead of new Long

2014-06-26 Thread Andrej Golovnin
Hi Paul, Seems fine to me (pending a full test run). Just a syntax niggle in the following, too many brackets: +++ new/src/share/classes/javax/management/modelmbean/RequiredModelMBean.java 2014-06-14 10:16:02.486298421 -0300 @@ -544,7 +544,7 @@ } // convert

Re: Long valueOf instead of new Long

2014-06-26 Thread Paul Sandoz
On Jun 26, 2014, at 3:29 PM, Andrej Golovnin andrej.golov...@gmail.com wrote: Hi Paul, Seems fine to me (pending a full test run). Just a syntax niggle in the following, too many brackets: +++ new/src/share/classes/javax/management/modelmbean/RequiredModelMBean.java 2014-06-14

Re: Process trees and termination

2014-06-26 Thread roger riggs
Hi David, On 6/25/2014 6:05 PM, David M. Lloyd wrote: On 06/25/2014 04:52 PM, roger riggs wrote: Hi, The next step for JEP 102, Process API Updates is to provide an API to enumerate the hierarchy of processes and to be able to terminate a process tree. This draft javadoc

Re: Process trees and termination

2014-06-26 Thread roger riggs
Hi Peter, Thanks for the comments... On 6/26/2014 4:55 AM, Peter Levart wrote: On 06/26/2014 12:05 AM, David M. Lloyd wrote: On 06/25/2014 04:52 PM, roger riggs wrote: Hi, The next step for JEP 102, Process API Updates is to provide an API to enumerate the hierarchy of processes and to be

Re: Process trees and termination

2014-06-26 Thread roger riggs
Hi Paul, Whether an interface or class, it is desirable that Process have the new capabilities of iterating over children and destroying the tree and other possible additions retrieving information about the process like the process name, start time, etc. If Process had a ProcessHandle then

Re: Ready for Review : 8042469 : Launcher changes for native memory tracking scalability enhancement

2014-06-26 Thread Neil Toda
Zhengyu Earlier creation of the environmental variable. http://cr.openjdk.java.net/~ntoda/8042469/webrev-07/ The new webrev [07] places the new code in the function SetJvmEnvironment() === . JLI_Launch() . InitLauncher() . CreateExecutionEnvironment() . CheckJvmType() .

Re: RFR [9] 8041972: Add improved parse/format methods for Long/Integer

2014-06-26 Thread Claes Redestad
On 06/25/2014 06:43 PM, Paul Sandoz wrote: On Jun 19, 2014, at 7:39 PM, Claes Redestad claes.redes...@oracle.com wrote: Hi, an updated webrev with reworked, public methods is available here: http://cr.openjdk.java.net/~redestad/8041972/webrev.8/ Reviews are yet again appreciated! I think if

Re: Ready for Review : 8042469 : Launcher changes for native memory tracking scalability enhancement

2014-06-26 Thread Zhengyu Gu
Hi Neil, There is still an issue. Apparently, you can NOT free the buffer for the environment variable. 678 char * pbuf = (char*)JLI_MemAlloc(pbuflen); 679 JLI_Snprintf(pbuf, pbuflen, %s%d=%s, NMT_Env_Name, JLI_GetPid(), value); 680 retval =

Re: Ready for Review : 8042469 : Launcher changes for native memory tracking scalability enhancement

2014-06-26 Thread Neil Toda
You are right. I found this upon looking: https://www.securecoding.cert.org/confluence/display/seccode/POS34-C.+Do+not+call+putenv()+with+a+pointer+to+an+automatic+variable+as+the+argument We moved to malloc since the launcher doesn't know about the value being passed with the native memory

Re: Ready for Review : 8042469 : Launcher changes for native memory tracking scalability enhancement

2014-06-26 Thread Zhengyu Gu
The possible values are: off, summary and detail Thanks, -Zhengyu On 6/26/2014 3:58 PM, Neil Toda wrote: You are right. I found this upon looking:

Re: Process trees and termination

2014-06-26 Thread Stuart Marks
On 6/26/14 7:23 AM, roger riggs wrote: On 6/26/2014 4:55 AM, Peter Levart wrote: - Will there be a guarantee that ProcessHandle objects returned from factory methods: [...] representing those processes that were started by ProcessBuilder API are actually the same Process objects that were

Re: ThreadLocalRandom clinit troubles

2014-06-26 Thread Doug Lea
Peter: Thanks very much for attacking the shocking impact/complexity of getting a few seed bits. On 06/25/2014 01:41 PM, Peter Levart wrote: Peeking around in the sun.security.provider package, I found there already is a minimal internal infrastructure for obtaining random seed. It's

Re: ThreadLocalRandom clinit troubles

2014-06-26 Thread Bradford Wetmore
On 6/26/2014 4:10 PM, Doug Lea wrote: This seems to be the best idea yet, assuming that people are OK with the changes to sun.security.provider.SeedGenerator and NativeSeedGenerator.java I've been meaning to review this thread, but have been chasing several urgent escalations. Brad

RFR: 8048207 : (xs) Collections.checkedQueue offer() calls add() on wrapped queue

2014-06-26 Thread Mike Duigou
Hello all; This changeset corrects an issue with the Collections.checkedQueue() utility method added in Java 8. The wrapper implementation incorrectly calls add() on the wrapped queue rather than offer(). I improved the existing unit test to include this condition (and converted it to

Re: JDK9 project: XML/JAXP Approachability / Ease of Use

2014-06-26 Thread huizhe wang
Thanks Michael! And, welcome to core-libs-dev! :-) On 6/26/2014 4:02 AM, Michael Kay wrote: Here are some quick thoughts about the state of XML support in the JDK: 1. XML Parser. The version of Xerces in the JDK has long been buggy, and no-one has been fixing the bugs. It needs to be replaced