Re: Request-Review: Remove two simple warnings in HttpsURLConnection.java

2011-08-27 Thread Xuelei Fan
The fix was integrated into OpenJDK 8. http://hg.openjdk.java.net/jdk8/tl/jdk/rev/e4729ad0d7b5 Sebastian, thank you for the contribution. Xuelei On 8/27/2011 4:27 PM, Xuelei Fan wrote: > On 8/27/2011 2:13 PM, Alan Bateman wrote: >> Xuelei Fan wrote: >>> Hi Sebastian, >>> >>> Thank you for your c

Re: Request-Review: Remove two simple warnings in HttpsURLConnection.java

2011-08-27 Thread Xuelei Fan
On 8/27/2011 2:13 PM, Alan Bateman wrote: > Xuelei Fan wrote: >> Hi Sebastian, >> >> Thank you for your contribution on OpenJDK. >> >> The update looks fine to me. Do you have a CR (Change Request) for this >> update? >> >> BTW, have you signed the Oracle Contribution Agreement [1]? I did not >> fi

Re: Request-Review: Remove two simple warnings in HttpsURLConnection.java

2011-08-26 Thread Alan Bateman
Xuelei Fan wrote: Hi Sebastian, Thank you for your contribution on OpenJDK. The update looks fine to me. Do you have a CR (Change Request) for this update? BTW, have you signed the Oracle Contribution Agreement [1]? I did not find you name in the OpenJDK people list. I don't think there is

Re: Request-Review: Remove two simple warnings in HttpsURLConnection.java

2011-08-26 Thread Xuelei Fan
Hi Sebastian, Thank you for your contribution on OpenJDK. The update looks fine to me. Do you have a CR (Change Request) for this update? BTW, have you signed the Oracle Contribution Agreement [1]? I did not find you name in the OpenJDK people list. Thanks, Xuelei [1] http://openjdk.java.net/c

Re: Request-Review: Remove two simple warnings in HttpsURLConnection.java

2011-08-26 Thread Alan Bateman
Sebastian Sickelmann wrote: Hi, is there someone who wants to review / support this simple warning removal? The webrev is: http://oss-patches.24.eu/openjdk8/Simple_Warning/ -- Sebastian* * This is JSSE so probably best if someone from the security area pushes this for Sebastian. -Alan.