On 3/29/12 4:25 AM, Michael McMahon wrote:
On 27/03/12 22:23, Michael McMahon wrote:
Could I get the following webrev for 7u4 reviewed please?
It is the same code change as for jdk 8
http://cr.openjdk.java.net/~michaelm/7134701/webrev.7u4.1/
Thanks,
Michael.
I've updated the webrev for this 7u4 change, based on the testing
yesterday.
It simplifies the original jdk 8 change (which will be updated via CR
7157665)
So, if an attempt to load a library whose name is X.dylib fails we
retry with the name X.jnilib
The following examples show the main cases:
System.loadLibrary("foo") -> libfoo.dylib, then libfoo.jnilib
(tried in each directory of the boot library path and user library
path respectively)
System.load("/abs/path/libfoo.dylib") -> /abs/path/libfoo.dylib, then
/abs/path/libfoo.jnilib
System.load("/abs/path/libfoo.jnilib") -> /abs/path/libfoo.jnilib
System.load("/abs/path/libfoo.bar") -> /abs/path/libfoo.bar
System.mapLibraryName("foo") -> "libfoo.dylib"
As before, this change only affects the behavior on Mac OS X.
http://cr.openjdk.java.net/~michaelm/7134701/webrev.7u4.2/
Nicely redone. Thumbs up!
make/java/java/FILES_java.gmk
src/share/classes/java/lang/ClassLoader.java
src/macosx/classes/java/lang/ClassLoaderHelper.java
src/solaris/classes/java/lang/ClassLoaderHelper.java
src/windows/classes/java/lang/ClassLoaderHelper.java
No comments on the above files.
Thanks,
Michael