Fix return code from alcMakeContextCurrent (#17591)

Fixes: #11708

Author: sbc100

src/library_openal.js

@@ -2298,10 +2298,10 @@ var LibraryOpenAL = {
   alcMakeContextCurrent: function(contextId) {
     if (contextId === 0) {
       AL.currentCtx = null;
-      return 0;
+    } else {
+      AL.currentCtx = AL.contexts[contextId];
     }
-    AL.currentCtx = AL.contexts[contextId];
-    return 1;
+    return 1 /* ALC_TRUE */;
   },
 
   alcGetContextsDevice__proxy: 'sync',

test/openal_error.c

@@ -8,19 +8,15 @@
 #include <stdio.h>
 #include <stdlib.h>
 #include <assert.h>
-#ifdef __EMSCRIPTEN__
-#include <emscripten.h>
 #include <AL/al.h>
 #include <AL/alc.h>
-#else
-#include "../system/include/AL/al.h"
-#include "../system/include/AL/alc.h"
-#endif
 
 ALCdevice* device = NULL;
 ALCcontext* context = NULL;
 
 int main(int argc, char* argv[]) {
+  ALCboolean ret;
+
   //
   // Setup the AL context.
   //
@@ -34,20 +30,20 @@ int main(int argc, char* argv[]) {
   assert(alcGetError(device) == ALC_NO_ERROR);
 
   context = alcCreateContext(device, NULL);
-  alcMakeContextCurrent(context);
+  ret = alcMakeContextCurrent(context);
+  assert(ret == ALC_TRUE);
 
   // Request an invalid enum to generate an AL error
   alGetFloat(0);
   assert(alGetError() == AL_INVALID_ENUM);
   // Check that the error is reset after reading it.
   assert(alGetError() == AL_NO_ERROR);
 
-  alcMakeContextCurrent(NULL);
+  ret = alcMakeContextCurrent(NULL);
+  assert(ret == ALC_TRUE);
+
   alcDestroyContext(context);
   alcCloseDevice(device);
-
-#ifdef __EMSCRIPTEN__
-    REPORT_RESULT(1);
-#endif
+  return 0;
 }
 

test/test_browser.py

@@ -2256,7 +2256,7 @@ def test_openal_error(self):
       ['--closure=1']
     ]:
       print(args)
-      self.btest('openal_error.c', expected='1', args=args)
+      self.btest_exit('openal_error.c', args=args)
 
   def test_openal_capture_sanity(self):
     self.btest('openal_capture_sanity.c', expected='0')