HTML-based mail is sneaky. You changed the text but not the actual link. http://cr.openjdk.java.net/~michaelm/7134701/webrev.7u4.2/
should work better. Everything still looks okay. -- Scott On Apr 2, 2012, at 10:23 AM, Michael McMahon wrote: > Sorry Scott, that was the wrong link. I mistakenly forwarded the original > email instead of the updated one.. > > This is the correct webrev: > > > > Thanks, > Michael. > > > On 02/04/12 17:20, Scott Kovatch wrote: >> >> On Apr 2, 2012, at 9:35 AM, Michael McMahon wrote: >> >>> This should have been reviewed on this list. So, I am forwarding the >>> request again. It hasn't changed since the original webrev was reviewed >>> on core-libs and the macos lists. >>> >>> It is the same code change as for jdk 8 >>> >>> http://cr.openjdk.java.net/~michaelm/7134701/webrev.7u4.1/ >> >> So in the end it looks like we will only support checking both .jnilib and >> .dylib on Mac: >> >> 1. When using System.loadLibrary() >> 2. The class calling System.loadLibrary() was NOT loaded from a custom class >> loader >> 3. isAbsolute is false, which implies System.load() was called. >> >> I saw the macosx discussion, and I think this is fine. If you're calling >> System.load() you know what file you want to load, and the extension >> shouldn't be inferred. Likewise, we shouldn't override the policy of a >> non-system class loader. >> >> I enthusiastically approve this change. :-) >> >> -- Scott K. >> >> ---------------------------------------- >> Scott Kovatch >> [email protected] >> Santa Clara/Pleasanton, CA >> >
