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 )?

--HG--
extra : convert_revision : svn%3Ac70aab31-4412-0410-b14c-859654838e24/trunk%40865

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;

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;