Re: RFR [9] Inet[4|6]Address class and fieldID initialization in networking native code

2014-02-04 Thread Alan Bateman
On 04/02/2014 16:12, Chris Hegarty wrote: And a link to the webrev that can people can access: http://cr.openjdk.java.net/~chegar/nativeInetCleanup/webrev/ -Chris. This is a good clean-up and looks good to me. -Alan.

Re: RFR [9] Inet[4|6]Address class and fieldID initialization in networking native code

2014-02-04 Thread Michael McMahon
Nice cleanup. Looks good, and thanks for catching the CHECK_NULL problem Michael On 04/02/14 16:12, Chris Hegarty wrote: And a link to the webrev that can people can access: http://cr.openjdk.java.net/~chegar/nativeInetCleanup/webrev/ -Chris. On 02/04/2014 04:01 PM, Chris Hegarty wrote: Hi

Re: RFR [9] Inet[4|6]Address class and fieldID initialization in networking native code

2014-02-04 Thread Chris Hegarty
And a link to the webrev that can people can access: http://cr.openjdk.java.net/~chegar/nativeInetCleanup/webrev/ -Chris. On 02/04/2014 04:01 PM, Chris Hegarty wrote: Hi, This change is mainly about clean up of the networking native code. There are a number of places that cache JNI global re

RFR [9] Inet[4|6]Address class and fieldID initialization in networking native code

2014-02-04 Thread Chris Hegarty
Hi, This change is mainly about clean up of the networking native code. There are a number of places that cache JNI global references to the Inet[4|6]Address classes, as well as caching their respective field IDs. It is also difficult to correctly handle propagation of possible JNI method inv