Fixed Brian's bugs

src/java/org/lwjgl/openal/BaseAL.java

@@ -74,38 +74,36 @@ public abstract class BaseAL {
 	 */
 	public void create() throws Exception {
 		if (created) {
-		  return;
+			return;
-    }
+		}
 
+		// need to pass path of possible locations of OAL to native side
+		String libpath = System.getProperty("java.library.path");
+		String seperator = System.getProperty("path.separator");
+		String libname;
 
-    // need to pass path of possible locations of OAL to native side
+		// libname is hardcoded atm - this will change in a near future...
-    String libpath = System.getProperty("java.library.path");
+		libname = (System.getProperty("os.name").toLowerCase().indexOf("windows") == -1) 
-    String seperator = System.getProperty("path.separator");
+		        ? "libopenal.so" 
-    String libname;
+		        : "OpenAL32.dll";     
 
-    // libname is hardcoded atm - this will change in a near future...
+		StringTokenizer st = new StringTokenizer(libpath, seperator);
-    libname = (System.getProperty("os.name").toLowerCase().indexOf("windows") == -1) 
-            ? "libopenal.so" 
-            : "OpenAL32.dll";     
             
-    StringTokenizer st = new StringTokenizer(libpath, seperator);
+		//create needed string array
+		String[] oalPaths = new String[st.countTokens()+1];
 
-    //create needed string array
+		//build paths
-    String[] oalPaths = new String[st.countTokens()+1];
+		for(int i=0;i<oalPaths.length - 1;i++) {
-    
+			oalPaths[i] = st.nextToken() + File.separator + libname; 
-    //build paths
+		}
-    for(int i=0;i<st.countTokens();i++) {
-      oalPaths[i] = st.nextToken() + File.separator + libname; 
-    }
-    
-    //add cwd path
-    oalPaths[oalPaths.length-1] = libname;
 
+		//add cwd path
+		oalPaths[oalPaths.length-1] = libname;
 		if (!nCreate(oalPaths)) {
 			throw new Exception("AL instance could not be created.");
-        }
+                }
 		created = true;
-        init();
+                init();
 	}
 	
 	/**

src/java/org/lwjgl/test/openal/ALTest.java

@@ -298,6 +298,10 @@ public class ALTest extends BasicTest {
         // Initialize Open AL manually
         //Open device
         device = alc.openDevice(null);
+	if (device == null) {
+		System.out.println("Could not create ALC device");
+		System.exit(-1);
+	}
         //Create context(s)
         context = alc.createContext(device, 0);
         //Set active context

src/native/common/extal.cpp

@@ -168,30 +168,26 @@ void LoadOpenAL(JNIEnv *env, jobjectArray oalPaths) {
 #ifdef _DEBUG
   printf("Found %d OpenAL paths\n", pathcount);
 #endif  
-#ifdef _WIN32
   for(int i=0;i<pathcount;i++) {
     jstring path = (jstring) env->GetObjectArrayElement(oalPaths, i);
+    const char *path_str = env->GetStringUTFChars(path, NULL);
 #ifdef _DEBUG
-  printf("Testing '%s'\n", env->GetStringUTFChars(path, NULL));
+    printf("Testing '%s'\n", path_str);
 #endif  
-    handleOAL = LoadLibrary(env->GetStringUTFChars(path, NULL));
+#ifdef _WIN32
-    if (handleOAL != NULL) {
+    handleOAL = LoadLibrary(path_str);
-#ifdef _DEBUG
-  printf("Found OpenAL at '%s'\n", env->GetStringUTFChars(path, NULL));
-#endif  
-      break;
-    }
-  }
 #endif
 #ifdef _X11
-  for(int i=0;i<pathcount;i++) {
+    handleOAL = dlopen(path_str, RTLD_LAZY);
-    jstring path = (jstring) env->GetObjectArrayElement(oalPaths, i);
+#endif
-    handleOAL = dlopen(env->GetStringUTFChars(path, NULL), RTLD_LAZY);
     if (handleOAL != NULL) {
+#ifdef _DEBUG
+      printf("Found OpenAL at '%s'\n", path_str);
+#endif  
       break;
     }
+    env->ReleaseStringUTFChars(path, path_str);
   }
-#endif
 }
 
 /**
@@ -199,10 +195,10 @@ void LoadOpenAL(JNIEnv *env, jobjectArray oalPaths) {
  */
 void UnLoadOpenAL() {
 #ifdef _WIN32
-  //FreeLibrary(handleOAL);
+  FreeLibrary(handleOAL);
 #endif
 #ifdef _X11
-   dlclose(handleOAL);
+  dlclose(handleOAL);
 #endif
 }