2nd round code review request: 7055363: jdk_security3 cleanup

2011-08-02 Thread Weijun Wang
Hi All http://cr.openjdk.java.net/~weijun/7055363/webrev.01/ *Attention*: please note that the jtreg mode for jdk_security3 is still RunOthervmBatch. Xuelei is now working on the JSSE tests. Once he is ready, he will change the mode to RunSamevmBatch. The code changes: * Still included in t

Re: [Fwd: Code review request: 7072353 JNDI libraries do not build with javac -Xlint:all -Werror]

2011-08-02 Thread Xuelei Fan
> Please review these JNDI changes. > Bug detail: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7072353 > webrev: http://cr.openjdk.java.net/~mduigou/7072353/0/webrev/ Thanks for your effort to make JNDI free of compile-warning. The work is hard, I appreciate it. 1. I noticed the copyright d

hg: jdk8/tl/jdk: 6978200: ServerSocket.toString include "port=0" in the returned String

2011-08-02 Thread chris . hegarty
Changeset: 809e8db0c142 Author:chegar Date: 2011-07-29 10:55 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/809e8db0c142 6978200: ServerSocket.toString include "port=0" in the returned String Summary: Removal of "port=0" from ServerSocket.toString method Reviewed-by: alanb, c

Re: [Fwd: Code review request: 7072353 JNDI libraries do not build with javac -Xlint:all -Werror]

2011-08-02 Thread Alan Bateman
Xuelei Fan wrote: : 1. I noticed the copyright date of a few files are unchanged, please update them before you push the changes. This has come up a few times but I don't think it is strictly required. Kelly or one of the release engineers run a script over the forest periodically to fix up

Re: 2nd round code review request: 7055363: jdk_security3 cleanup

2011-08-02 Thread Alan Bateman
Weijun Wang wrote: Hi All http://cr.openjdk.java.net/~weijun/7055363/webrev.01/ I went through the changes to the tests and they look okay to me. It's good to see so many tests removed from the problem list and the tests updated to work in samevm or agentvm mode. Is there a bug created for t

Re: [Fwd: Code review request: 7072353 JNDI libraries do not build with javac -Xlint:all -Werror]

2011-08-02 Thread Xuelei Fan
On Aug 3, 2011, at 12:11 AM, Alan Bateman wrote: > Xuelei Fan wrote: >> : >> 1. I noticed the copyright date of a few files are unchanged, please >> update them before you push the changes. >> > This has come up a few times but I don't think it is strictly required. Kelly > or one of the rel

Re: [Fwd: Code review request: 7072353 JNDI libraries do not build with javac -Xlint:all -Werror]

2011-08-02 Thread Alexandre Boulgakov
Thanks for reviewing! See my responses inline. I'll wait on sending another webrev until I've received the rest of your comments. -Sasha On 8/2/2011 2:19 AM, Xuelei Fan wrote: Please review these JNDI changes. Bug detail: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7072353 webrev: htt

Re: [Fwd: Code review request: 7072353 JNDI libraries do not build with javac -Xlint:all -Werror]

2011-08-02 Thread Alexandre Boulgakov
Thanks for reviewing! Please see my responses inline. I'll wait on sending another webrev until I've received the rest of your comments. -Sasha On 8/2/2011 2:19 AM, Xuelei Fan wrote: Please review these JNDI changes. Bug detail: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7072353 webr

Re: 2nd round code review request: 7055363: jdk_security3 cleanup

2011-08-02 Thread Weijun Wang
On 08/03/2011 12:43 AM, Alan Bateman wrote: Weijun Wang wrote: Hi All http://cr.openjdk.java.net/~weijun/7055363/webrev.01/ I went through the changes to the tests and they look okay to me. It's good to see so many tests removed from the problem list and the tests updated to work in samevm o

Re: [Fwd: Code review request: 7072353 JNDI libraries do not build with javac -Xlint:all -Werror]

2011-08-02 Thread Xuelei Fan
com/sun/jndi/toolkit/dir/SearchFilter.java 451 for (NamingEnumeration ve = attr.getAll(); 452 ve.hasMore(); 453) { The update is OK. But the coding style looks uncomfortable. Would you mind change it to use for-each style? . javax/naming/directory/BasicAtt

Re: [Fwd: Code review request: 7072353 JNDI libraries do not build with javac -Xlint:all -Werror]

2011-08-02 Thread Dr Andrew John Hughes
On 17:11 Tue 02 Aug , Alan Bateman wrote: > Xuelei Fan wrote: > > : > > 1. I noticed the copyright date of a few files are unchanged, please > > update them before you push the changes. > > > This has come up a few times but I don't think it is strictly required. > Kelly or one of the relea