Modify

Ticket #727 (closed defect: invalid)

Opened 6 years ago

Last modified 6 years ago

mythmusic compile error in aacdecoder.cpp

Reported by: mythtv@… Owned by: ijr
Priority: minor Milestone: unknown
Component: mythmusic Version:
Severity: medium Keywords:
Cc: Ticket locked: no

Description

make[2]: Entering directory `/root/mythplugins/mythmusic/mythmusic'
g++ -c -pipe -march=pentiumpro -mmmx -Wall -W -O3 -Wall -Wno-switch -fomit-frame-pointer -I/usr/include/SDL -D_REENTRANT -D_REENTRANT -DPIC -fPIC  -D_GNU_SOURCE -DPREFIX=\"/usr/local\" -DMMX -Di386 -DUSING_DBOX2 -DHAVE_DVDNAV -D_FILE_OFFSET_BITS=64 -DQT_NO_DEBUG -DQT_THREAD_SUPPORT -DQT_PLUGIN -DQT_SHARED -DQT_TABLET_SUPPORT -I/usr/qt/3/mkspecs/linux-g++ -I. -I/usr/local/include -I/usr/kde/3.3/include -I/usr/local/include -I/usr/include -I/usr/include -I/usr/include/cdda -I/usr/qt/3/include -I/usr/include -o aacdecoder.o aacdecoder.cpp
aacdecoder.cpp: In member function `bool aacDecoder::initializeMP4()':
aacdecoder.cpp:298: error: invalid conversion from `long unsigned int*' to `
   uint32_t*'

myth mythmusic # gcc -v
Reading specs from /usr/lib/gcc-lib/i686-pc-linux-gnu/3.3.5-20050130/specs
Configured with: /var/tmp/portage/gcc-3.3.5.20050130-r1/work/gcc-3.3.5/configure --enable-version-specific-runtime-libs --prefix=/usr --bindir=/usr/i686-pc-linux-gnu/gcc-bin/3.3.5-20050130 --includedir=/usr/lib/gcc-lib/i686-pc-linux-gnu/3.3.5-20050130/include --datadir=/usr/share/gcc-data/i686-pc-linux-gnu/3.3.5-20050130 --mandir=/usr/share/gcc-data/i686-pc-linux-gnu/3.3.5-20050130/man --infodir=/usr/share/gcc-data/i686-pc-linux-gnu/3.3.5-20050130/info --with-gxx-include-dir=/usr/lib/gcc-lib/i686-pc-linux-gnu/3.3.5-20050130/include/g++-v3 --host=i686-pc-linux-gnu --build=i686-pc-linux-gnu --disable-altivec --enable-nls --without-included-gettext --with-system-zlib --disable-checking --disable-werror --disable-libunwind-exceptions --disable-multilib --disable-libgcj --enable-languages=c,c++,f77 --enable-shared --enable-threads=posix --enable-__cxa_atexit --enable-clocale=gnu
Thread model: posix
gcc version 3.3.5-20050130 (Gentoo 3.3.5.20050130-r1, ssp-3.3.5.20050130-1, pie-8.7.7.1)

Changing sample_rate to a uint32_t fixed the compile error, but I have no AAC recordings to test functionality on.

Attachments

mythmusic-compile.patch (426 bytes) - added by mythtv@… 6 years ago.
change sample_rate to uint32_t
mythtv-broken_aac.diff (567 bytes) - added by giuly@… 6 years ago.
Cast to (uint32_t*)
mythtv-broken_aac-platform_detection.diff (459 bytes) - added by giuly@… 6 years ago.
Uses uint32_t for x86_64, unsigned long for others

Change History

Changed 6 years ago by mythtv@…

change sample_rate to uint32_t

comment:2 Changed 6 years ago by paulh

  • Status changed from new to closed
  • Resolution set to invalid

It compiles fine with the standard unpatched ffaad2 files. The problem is caused by using a patched version that has been changed to make it compatible with 64bit machines.

I think your patch will allow mythmusic to compile with the patched faad2 but will break it for everyone using the standard faad2.

FAAD2: http://www.audiocoding.com/modules/mydownloads/

Changed 6 years ago by giuly@…

Cast to (uint32_t*)

comment:3 Changed 6 years ago by anonymous

  • Status changed from closed to reopened
  • Resolution invalid deleted

comment:4 Changed 6 years ago by ijr

  • Status changed from reopened to closed
  • Resolution set to invalid

That's functionally identical to the patch that was rejected before.

Changed 6 years ago by giuly@…

Uses uint32_t for x86_64, unsigned long for others

View

Add a comment

Modify Ticket

Action
as closed
The resolution will be deleted. Next status will be 'new'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.