Compiling aacgain on OSX

There is a nifty tool out there to adjust your mp3/aac collection to a similar volume …without any re-compression or data loss. Cool! But if you have ever tried to compile ~~aacgain~~1.5 on OSX (Intel) yourself you will notice that the compilation of libfaad fails.

In file included from /usr/include/math.h:28,
                from common.h:341,
                from bits.c:28:
/usr/include/architecture/i386/math.h:378: error: conflicting types for 'lrintf'
common.h:308: error: previous definition of 'lrintf' was here

This could probably be fixed by some configure magic or checking for OSX in the header file. As a quick fix I’ve just commented it out.

--- aacgain-1.5/faad2/libfaad/common.h  2004-09-08 11:43:12.000000000 +0200
+++ aacgain-1.5.fixed/faad2/libfaad/common.h    2006-10-20 01:01:30.000000000 +0200
@@ -302,6 +302,7 @@
         return i;
     }
   #elif (defined(__i386__) && defined(__GNUC__))
+/*
     #define HAS_LRINTF
     // from http://www.stereopsis.com/FPU.html
     static INLINE int lrintf(float f)
@@ -314,6 +315,7 @@
             : "m" (f));
         return i;
     }
+*/
   #endif

Would be really nice to make this package available through fink.

20. October 2006 | c, music, osx

Logan

Hey there,

What you need to do is remove the "static" and replace with "long":

long INLINE int lrintf(float f)

I'm compiling on a Core 2 Duo MacBook, and worked perfectly.
harijay

Hi what Johnny said did not work for me ..I get the error
In file included from drm_dec.c:37:
common.h:329: error: static declaration of 'lrintf' follows non-static declaration
make[2]: *** [drm_dec.lo] Error 1
make[1]: *** [all-recursive] Error 1
make: *** [all] Error 2
Johnny

All you have to do is add the word "long":

#define HAS_LRINTF
// from http://www.stereopsis.com/FPU....
static long INLINE int lrintf(float f)

Torsten Curdt’s weblog

Compiling aacgain on OSX