On 14/09/2012 19:22, John Zavgren wrote:
:
The proposed changes to the code
(http://cr.openjdk.java.net/~chegar/7193520/webrev.00/) eliminate distracting
dead wood, and it makes it run (slightly) faster, because the run time checks
are eliminated.
It's good to remove this old and long unused code. There is also some
Linux 2.4 specific code in there too and maybe that should be removed too.
I agree with Dmitry's comment that the braces aren't needed at L318 +
324 of PlainDatagramSocketImpl.c.
-Alan.
