hg: jdk8/tl/jdk: 7068321: Support TLS Server Name Indication (SNI) Extension in JSSE Server

2012-10-18 Thread xuelei . fan
Changeset: 3f62cfc4e83d Author:xuelei Date: 2012-10-18 01:14 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/3f62cfc4e83d 7068321: Support TLS Server Name Indication (SNI) Extension in JSSE Server Reviewed-by: mullan, weijun, wetmore ! src/share/classes/javax/net/ssl/Extended

Change to: ./src/solaris/native/java/net/PlainDatagramSocketImpl.c

2012-10-18 Thread John Zavgren
Greetings: I'm proposing the following change: http://cr.openjdk.java.net/~khazra/john/8000206/webrev.00/ because it simplifies the code by eliminating an unnecessary union data structure. This change eliminates a false positive from our static code analyzer: parfait. This modification doesn't

Re: Change to: ./src/solaris/native/java/net/PlainDatagramSocketImpl.c

2012-10-18 Thread Dmitry Samersoff
John, Looks good for me. -Dmitry On 2012-10-19 00:59, John Zavgren wrote: > Greetings: > I'm proposing the following change: > > http://cr.openjdk.java.net/~khazra/john/8000206/webrev.00/ > > because it simplifies the code by eliminating an unnecessary union data > structure. This change elim

Re: Change to: ./src/solaris/native/java/net/PlainDatagramSocketImpl.c

2012-10-18 Thread Kurchi Hazra
Hi John, This change looks good to me. Thanks, - Kurchi On 18.10.2012 14:18, Dmitry Samersoff wrote: John, Looks good for me. -Dmitry On 2012-10-19 00:59, John Zavgren wrote: Greetings: I'm proposing the following change: http://cr.openjdk.java.net/~khazra/john/8000206/webrev.00/ becau