After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 305149 - faad plugin in 0.8.8.2 pre-release doesn't compile on amd64
faad plugin in 0.8.8.2 pre-release doesn't compile on amd64
Status: RESOLVED INVALID
Product: GStreamer
Classification: Platform
Component: gst-plugins
0.8.8
Other Linux
: Normal normal
: NONE
Assigned To: GStreamer Maintainers
GStreamer Maintainers
: 319795 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2005-05-23 00:38 UTC by Andy Neitzke
Modified: 2005-10-27 13:33 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Andy Neitzke 2005-05-23 00:38:56 UTC
Distribution/Version: Gentoo

Attempting to compile the faad plugin in the 0.8.8.2 pre-release fails on 
amd64.  The trouble is that "samplerate" in gstfaad.c is defined as "gulong", 
which is 64-bit on amd64, while in faad.h it is defined as "uint32_t".  I'm not 
sure what is the most correct way of fixing this, but changing "gulong" to 
"uint32_t" on lines 304 and 677 of gstfaad.c allowed the plugin to compile for 
me.
Comment 1 Ronald Bultje 2005-06-03 09:07:15 UTC
In my copy, it is defined as unsigned long:

/* Init the library based on info from the AAC file (ADTS/ADIF) */
long FAADAPI faacDecInit(faacDecHandle hDecoder,
                        unsigned char *buffer,
                        unsigned long buffer_size,
                        unsigned long *samplerate,
                        unsigned char *channels);

/* Init the library using a DecoderSpecificInfo */
char FAADAPI faacDecInit2(faacDecHandle hDecoder, unsigned char *pBuffer,
                         unsigned long SizeOfDecoderSpecificInfo,
                         unsigned long *samplerate, unsigned char *channels);

What version of libfaad2?
Comment 2 Andy Neitzke 2005-06-09 10:25:55 UTC
It's libfaad2 version 2.0 -- but it turns out that the Gentoo packages for some   
reason include a patch which changes the line you listed, among others:   
   
  long FAADAPI faacDecInit(faacDecHandle hDecoder,   
-                        unsigned char *buffer,   
-                        unsigned long buffer_size,   
-                        unsigned long *samplerate,   
-                        unsigned char *channels);   
+                         uint8_t *buffer,   
+                         uint32_t buffer_size,   
+                         uint32_t *samplerate,   
+                         uint8_t *channels);   
   
So I guess this bug is invalid.  Sorry for the inconvenience.   
Comment 3 Tim-Philipp Müller 2005-10-27 13:33:09 UTC
*** Bug 319795 has been marked as a duplicate of this bug. ***