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

Kurchi Hazra kurchi.subhra.hazra at oracle.com
Thu Oct 18 15:27:38 PDT 2012


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/
>>
>> 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 change any of the externally-visible behavior in the procedure.
>>
>> I look forward to your comments and suggestions.
>>
>> Thanks!
>> John Zavgren
>> john.zavgren at oracle.com
>>
>

-- 
-Kurchi




More information about the net-dev mailing list