MythMusic compile problem with --enable-aac

[otto at kolsi dot fi]

If mythplugins is configured with --enable-aac, following compile error occurs. System is Fedora 9.

g++ -c -pipe -march=pentium4 -Wall -Wno-switch -Wpointer-arith -Wredundant-decls -Wno-non-virtual-dtor -funit-at-a-time -D__STDC_CONSTANT_MACROS -fomit-frame-pointer -O3 -fomit-frame-pointer -I/usr/include/SDL -D_GNU_SOURCE=1 -D_REENTRANT -D_REENTRANT -Wall -W -DPIC -fPIC -D_GNU_SOURCE -DPREFIX="/usr/local" -DMMX -Di386 -D_FILE_OFFSET_BITS=64 -DQT_NO_DEBUG -DQT_PLUGIN -DQT_QT3SUPPORT_LIB -DQT3_SUPPORT -DQT_SQL_LIB -DQT_XML_LIB -DQT_OPENGL_LIB -DQT_GUI_LIB -DQT_CORE_LIB -I/usr/lib/qt4/mkspecs/linux-g++ -I. -I/usr/include/QtCore -I/usr/include/QtCore -I/usr/include/QtGui -I/usr/include/QtGui -I/usr/include/QtOpenGL -I/usr/include/QtOpenGL -I/usr/include/QtXml -I/usr/include/QtXml -I/usr/include/QtSql -I/usr/include/QtSql -I/usr/include/Qt3Support -I/usr/include/Qt3Support -I/usr/include -I/usr/local/include -I/usr/local/include -I/usr/include -I/usr/local/include/mythtv -I/usr/include/libvisual-0.4 -I/usr/include/taglib -I/usr/include/cdda -I/usr/X11R6/include -I. -I. -o aacdecoder.o aacdecoder.cpp
aacdecoder.cpp: In member function ‘bool aacDecoder::initializeMP4()’:
aacdecoder.cpp:317: error: invalid conversion from ‘uint32_t*’ to ‘long unsigned int*’
aacdecoder.cpp:317: error:   initializing argument 4 of ‘char NeAACDecInit2(void*, unsigned char*, long unsigned int, long unsigned int*, unsigned char*)’
make[2]: *** [aacdecoder.o] Error 1
make[2]: Leaving directory `/mnt/myth/trunk/mythplugins/mythmusic/mythmusic'
make[1]: *** [sub-mythmusic-make_default] Error 2
make[1]: Leaving directory `/mnt/myth/trunk/mythplugins/mythmusic'
make: *** [sub-mythmusic-make_default] Error 2

Without --enable-aac option mythplugins compile without errors.

[stuartm]

I'll be removing the aac decoder from trunk very soon, but if someone submits a patch for -fixes we'll fix it there.

[otto at kolsi dot fi]

Replying to stuartm:

I'll be removing the aac decoder from trunk very soon, but if someone submits a patch for -fixes we'll fix it there.

--enable-aac works fine with -fixes in the same machine, so maybe this ticket can be closed then?

[Paul Mason <latepaul@…>]

second patch replaces the first one which had a diff due to a typo.

[Britney Fransen <britney.fransen@…>]

Using .21-fixes and --enable-aac I just saw this same error. Applying the second patch fixed the error and it then compiled without error.