[Openal-devel] make two warnings to one in alc_context

Prakash Punnoor prakash at punnoor.de
Mon Oct 30 02:38:16 PST 2006


Hi,

what do you think about this? Do you think it will kill readability too much 
or think it is OK?


Index: src/alc/alc_context.c
===================================================================
--- src/alc/alc_context.c       (Revision 1403)
+++ src/alc/alc_context.c       (Arbeitskopie)
@@ -1288,12 +1288,10 @@
 alcGetProcAddress( ALCdevice *device, const ALCchar *funcName )
 {
        AL_funcPtr value;
-       if (getStandardProcAddress(&value, funcName) == ALC_TRUE) {
+       if ((getStandardProcAddress(&value, funcName) == ALC_TRUE) ||
+               (getExtensionProcAddress(&value, device, funcName) == 
ALC_TRUE)) {
                return (void *)value; /* NOTE: The cast is not valid ISO C! */
        }
-       if (getExtensionProcAddress(&value, device, funcName) == ALC_TRUE) {
-               return (void *)value; /* NOTE: The cast is not valid ISO C! */
-       }
        _alcSetError( ALC_INVALID_VALUE );
        return NULL;
 }

-- 
(°=                 =°)
//\ Prakash Punnoor /\\
V_/                 \_V
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://opensource.creative.com/pipermail/openal-devel/attachments/20061030/70077a2b/attachment.bin


More information about the Openal-devel mailing list