Re: Request for review - change two include header files according to POSIX.1-2008

2011-10-13 Thread Charles Lee
On 10/13/2011 01:11 PM, David Holmes wrote: Hi Charles, On 13/10/2011 2:54 PM, Charles Lee wrote: Thanks Alan. Below is the patch I am failed to attach. It is trivial... Do you need someone to sponsor this for you, or are you able to drive this via other IBM folk that can generate webrevs

Re: Request for review - change two include header files according to POSIX.1-2008

2011-10-13 Thread Chris Hegarty
I think I seen in a previous mail that you didn't have access to Solaris machines? I'll grab your patch, apply to a latest jdk8 TL repo and do some sanity builds on Solaris if you like? -Chris. On 10/13/11 01:43 PM, Alan Bateman wrote: Neil Richards wrote: : Will there need to be two bug

Re: Request for review - change two include header files according to POSIX.1-2008

2011-10-13 Thread Neil Richards
On Thu, 2011-10-13 at 14:03 +0100, Chris Hegarty wrote: I think I seen in a previous mail that you didn't have access to Solaris machines? I'll grab your patch, apply to a latest jdk8 TL repo and do some sanity builds on Solaris if you like? -Chris. Hi Chris, Thank you, that would be

hg: jdk8/tl/jdk: 7099228: Use a PKCS11 config attribute to control encoding of an EC point

2011-10-13 Thread vincent . x . ryan
Changeset: 2b27e14a4c82 Author:vinnie Date: 2011-10-13 12:00 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/2b27e14a4c82 7099228: Use a PKCS11 config attribute to control encoding of an EC point Reviewed-by: valeriep, mullan !

Re: Codereview request for 7096080: UTF8 update and new CESU-8 charset

2011-10-13 Thread Ulf Zibis
Am 11.10.2011 19:49, schrieb Xueming Shen: I don't know which one is better, I did a run on private static boolean op1(int b) { return (b 6) != -2; } private static boolean op2(int b) { return (b 0xc0) != 0x80; } private static boolean op3(byte b) {

Re: Request for review - change two include header files according to POSIX.1-2008

2011-10-13 Thread Chris Hegarty
Neil, All builds complete with your patches (on top of the latest JDK8 TL repo) from : http://cr.openjdk.java.net/~ngmr/ojdk-243.1/webrev.01/ http://cr.openjdk.java.net/~ngmr/ojdk-243.2/webrev.00/ solaris_sparc_5.10-product solaris_sparcv9_5.10-product solaris_i586_5.10-product

hg: jdk8/tl/jdk: 2 new changesets

2011-10-13 Thread sean . mullan
Changeset: 01615d3e74ed Author:mullan Date: 2011-10-13 13:50 -0400 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/01615d3e74ed 6953295: Move few sun.security.{util, x509, pkcs} classes used by keytool/jarsigner to another package Reviewed-by: mchung !

Re: Codereview request for 7096080: UTF8 update and new CESU-8 charset

2011-10-13 Thread Xueming Shen
On 10/13/2011 09:55 AM, Ulf Zibis wrote: Am 11.10.2011 19:49, schrieb Xueming Shen: I don't know which one is better, I did a run on private static boolean op1(int b) { return (b 6) != -2; } private static boolean op2(int b) { return (b 0xc0) != 0x80; }

Re: Patch to Throwable and StackTraceElement (reduced CPU usage)

2011-10-13 Thread Mike Skells
Hi Patrick, David, David - I tried your suggestions, and I agree it must be faster, but it is in the noise of my test env. not very satisfactory Patrick - Unfortunately Appendable doesn't provide all of the interface that you need. All of the methods throw IOException and you cannot

Re: Patch to Throwable and StackTraceElement (reduced CPU usage)

2011-10-13 Thread Patrick Reinhart
Am 14.10.11 01:06, schrieb Mike Skells: Hi Patrick, David, David - I tried your suggestions, and I agree it must be faster, but it is in the noise of my test env. not very satisfactory Patrick - Unfortunately Appendable doesn't provide all of the interface that you need. All of the