hg: jdk7/tl/jdk: 6217210: RFE: Support for Cp833 in 1.4.2

2010-11-17 Thread xueming . shen
Changeset: ce757906302f Author:sherman Date: 2010-11-17 21:33 -0800 URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/ce757906302f 6217210: RFE: Support for Cp833 in 1.4.2 Summary: Forward port the Cp833 charset Reviewed-by: poonam ! make/sun/nio/cs/FILES_java.gmk + make/tools/Charse

Re: Code Review Request 6203816 and 6720456

2010-11-17 Thread Valerie (Yu-Ching) Peng
Thanks for the lightning fast review! TBD means "to be determined at runtime". Different machines w/ different versions of Solaris may support different key sizes. So, I use TBD to indicate the key sizes which may only be supported by the newer versions of Solairs release. I thought you are

Re: Code Review Request 6203816 and 6720456

2010-11-17 Thread Weijun Wang
On 11/18/2010 07:31 AM, Valerie (Yu-Ching) Peng wrote: Hi, Max, Can you please help reviewing the following two regression test fixes? 6203816: Can not run test/java/security/Security/ClassLoaderDeadlock.sh from the command line Webrev: http://cr.openjdk.java.net/~valeriep/6203816/webrev.00/

Code Review Request for 6604496: Support for CKM_AES_CTR (counter mode)

2010-11-17 Thread Valerie (Yu-Ching) Peng
Vinnie, Can you please help reviewing the following fix for supporting AES with CTR mode? 6604496: Support for CKM_AES_CTR (counter mode) Webrev: http://cr.openjdk.java.net/~valeriep/6604496/webrev.00/ Thanks, Valerie

Code Review Request 6203816 and 6720456

2010-11-17 Thread Valerie (Yu-Ching) Peng
Hi, Max, Can you please help reviewing the following two regression test fixes? 6203816: Can not run test/java/security/Security/ClassLoaderDeadlock.sh from the command line Webrev: http://cr.openjdk.java.net/~valeriep/6203816/webrev.00/ I noticed that Deadlock2.sh would fail when the TESTJAVA

hg: jdk7/tl/jdk: 6615506: (fmt spec) Date/Time conversion table missing column for 'Z'

2010-11-17 Thread xueming . shen
Changeset: 59d10b97be7c Author:sherman Date: 2010-11-17 15:10 -0800 URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/59d10b97be7c 6615506: (fmt spec) Date/Time conversion table missing column for 'Z' Summary: added the column entry back in Reviewed-by: alanb ! src/share/classes/java

hg: jdk7/tl/langtools: 7000973: isBogus needs to be called on the to-be-returned entry, not on the current entry

2010-11-17 Thread jonathan . gibbons
Changeset: abaceae7c9f8 Author:jjg Date: 2010-11-17 15:07 -0800 URL: http://hg.openjdk.java.net/jdk7/tl/langtools/rev/abaceae7c9f8 7000973: isBogus needs to be called on the to-be-returned entry, not on the current entry Reviewed-by: jjg Contributed-by: jan.lah...@oracle.com ! src

Re: 6995424 Code Review Request

2010-11-17 Thread Mandy Chung
On 11/17/10 12:41 PM, Sean Mullan wrote: On 11/17/2010 02:34 PM, Mandy Chung wrote: Hi Sean, On 11/16/10 5:56 AM, Sean Mullan wrote: http://cr.openjdk.java.net/~mullan/6995424/webrev.00/ Policy.java: I was initially confused with the meaning of 'compatPolicy'. I thought that when 'compatPo

Re: 6995424 Code Review Request

2010-11-17 Thread Sean Mullan
On 11/17/2010 02:34 PM, Mandy Chung wrote: Hi Sean, On 11/16/10 5:56 AM, Sean Mullan wrote: http://cr.openjdk.java.net/~mullan/6995424/webrev.00/ Policy.java: I was initially confused with the meaning of 'compatPolicy'. I thought that when 'compatPolicy' is set to true, it means that the dep

Re: 6995424 Code Review Request

2010-11-17 Thread Mandy Chung
Hi Sean, On 11/16/10 5:56 AM, Sean Mullan wrote: http://cr.openjdk.java.net/~mullan/6995424/webrev.00/ Policy.java: I was initially confused with the meaning of 'compatPolicy'. I thought that when 'compatPolicy' is set to true, it means that the deprecated javax.security.auth.Policy c

hg: jdk7/tl/jdk: 2 new changesets

2010-11-17 Thread michael . x . mcmahon
Changeset: f9dbb7d2e8a3 Author:michaelm Date: 2010-11-17 14:29 + URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/f9dbb7d2e8a3 6725892: Http server stability issues Reviewed-by: chegar ! src/share/classes/com/sun/net/httpserver/HttpsConfigurator.java ! src/share/classes/com/sun/