iOS: NativeLibrary: Refine iOS, use proper alt system path for iOS+OSX, relax closeLibrary(..)

2 files changed, 38 insertions, 28 deletions

diff --git a/src/java/com/jogamp/common/os/DynamicLibraryBundle.java b/src/java/com/jogamp/common/os/DynamicLibraryBundle.java
index a3d6198..fee3c01 100644
--- a/src/java/com/jogamp/common/os/DynamicLibraryBundle.java
+++ b/src/java/com/jogamp/common/os/DynamicLibraryBundle.java
@@ -282,7 +282,7 @@ public class DynamicLibraryBundle implements DynamicLookupHelper {
                     }
                 } else {
                     if( null == dynLinkGlobal ) {
-                        dynLinkGlobal = lib.getDynamicLinker();
+                        dynLinkGlobal = lib.dynamicLinker();
                     }
                     nativeLibraries.add(lib);
                     toolLibLoaded[i]=true;
diff --git a/src/java/com/jogamp/common/os/NativeLibrary.java b/src/java/com/jogamp/common/os/NativeLibrary.java
index 39d1475..1b05700 100644
--- a/src/java/com/jogamp/common/os/NativeLibrary.java
+++ b/src/java/com/jogamp/common/os/NativeLibrary.java
@@ -89,12 +89,17 @@ public final class NativeLibrary implements DynamicLookupHelper {
         break;
 
       case MACOS:
-      case IOS:
         prefixes = new String[] { "lib" };
         suffixes = new String[] { ".dylib", ".jnilib" };
         isOSX = true;
         break;
 
+      case IOS:
+        prefixes = new String[] { "lib" };
+        suffixes = new String[] { ".dylib" };
+        isOSX = true;
+        break;
+
       /*
       case ANDROID:
       case FREEBSD:
@@ -211,29 +216,7 @@ public final class NativeLibrary implements DynamicLookupHelper {
                                                        loader);
     Platform.initSingleton(); // loads native gluegen-rt library
 
-    final DynamicLinker dynLink;
-    switch (PlatformPropsImpl.OS_TYPE) {
-      case WINDOWS:
-        dynLink = new WindowsDynamicLinkerImpl();
-        break;
-
-      case MACOS:
-      case IOS:
-        dynLink = new MacOSXDynamicLinkerImpl();
-        break;
-
-      case ANDROID:
-        if( PlatformPropsImpl.CPU_ARCH.is32Bit ) {
-            dynLink = new BionicDynamicLinker32bitImpl();
-        } else {
-            dynLink = new BionicDynamicLinker64BitImpl();
-        }
-        break;
-
-      default:
-        dynLink = new PosixDynamicLinkerImpl();
-        break;
-    }
+    final DynamicLinker dynLink = getDynamicLinker();
 
     // Iterate down these and see which one if any we can actually find.
     for (final Iterator<String> iter = possiblePaths.iterator(); iter.hasNext(); ) {
@@ -312,7 +295,34 @@ public final class NativeLibrary implements DynamicLookupHelper {
     return dynLink.lookupSymbolGlobal(funcName);
   }
 
-  /* pp */ final DynamicLinker getDynamicLinker() { return dynLink; }
+  /* pp */ final DynamicLinker dynamicLinker() { return dynLink; }
+
+  /* pp */ static DynamicLinker getDynamicLinker() {
+      final DynamicLinker dynLink;
+      switch (PlatformPropsImpl.OS_TYPE) {
+          case WINDOWS:
+              dynLink = new WindowsDynamicLinkerImpl();
+              break;
+
+          case MACOS:
+          case IOS:
+              dynLink = new MacOSXDynamicLinkerImpl();
+              break;
+
+          case ANDROID:
+              if( PlatformPropsImpl.CPU_ARCH.is32Bit ) {
+                  dynLink = new BionicDynamicLinker32bitImpl();
+              } else {
+                  dynLink = new BionicDynamicLinker64BitImpl();
+              }
+              break;
+
+          default:
+              dynLink = new PosixDynamicLinkerImpl();
+              break;
+      }
+      return dynLink;
+  }
 
   /** Retrieves the low-level library handle from this NativeLibrary
       object. On the Windows platform this is an HMODULE, and on Unix
@@ -433,9 +443,9 @@ public final class NativeLibrary implements DynamicLookupHelper {
         // Add probable Mac OS X-specific paths
         if ( isOSX ) {
             // Add historical location
-            addPaths("/Library/Frameworks/" + libName + ".Framework", baseNames, paths);
+            addPaths("/Library/Frameworks/" + libName + ".framework", baseNames, paths);
             // Add current location
-            addPaths("/System/Library/Frameworks/" + libName + ".Framework", baseNames, paths);
+            addPaths("/System/Library/Frameworks/" + libName + ".framework", baseNames, paths);
         }
     }