Date: Sun, 29 Feb 2004 20:28:27 +0200

From: Martin_Storsj Subject: Slight bug in ESD and aRts When I experimented with the ALSA-patch, I found a slight bug in the Load{ESD,ARTS}Library-functions. The check of whether a function pointer was correctly loaded looks like this right now: *esd_functions[i].func = SDL_LoadFunction(esd_handle, esd_functions[i].name); if ( ! esd_functions[i].func ) Isn't that supposed to be ( ! *esd_functions[i].func )?
libsdl-org · Mar 2, 2004 · 1ba3e4c · 1ba3e4c
1 parent 99d06e3
commit 1ba3e4c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/audio/arts/SDL_artsaudio.c b/src/audio/arts/SDL_artsaudio.c
@@ -106,7 +106,7 @@ static int LoadARTSLibrary(void)
 		retval = 0;
 		for ( i=0; i<SDL_TABLESIZE(arts_functions); ++i ) {
 			*arts_functions[i].func = SDL_LoadFunction(arts_handle, arts_functions[i].name);
-			if ( ! arts_functions[i].func ) {
+			if ( !*arts_functions[i].func ) {
 				retval = -1;
 				UnloadARTSLibrary();
 				break;

diff --git a/src/audio/esd/SDL_esdaudio.c b/src/audio/esd/SDL_esdaudio.c
@@ -101,7 +101,7 @@ static int LoadESDLibrary(void)
 		retval = 0;
 		for ( i=0; i<SDL_TABLESIZE(esd_functions); ++i ) {
 			*esd_functions[i].func = SDL_LoadFunction(esd_handle, esd_functions[i].name);
-			if ( ! esd_functions[i].func ) {
+			if ( !*esd_functions[i].func ) {
 				retval = -1;
 				UnloadESDLibrary();
 				break;