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 133064 - ext/a52dec/gsta52dec.c and uint32_t, uint8_t
ext/a52dec/gsta52dec.c and uint32_t, uint8_t
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins
0.7.3
Other Solaris
: Normal blocker
: 0.7.4
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2004-01-31 15:01 UTC by Stefan Sauer (gstreamer, gtkdoc dev)
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Stefan Sauer (gstreamer, gtkdoc dev) 2004-01-31 15:01:23 UTC
in ext/a52dec/gsta52dec.c you use :

#ifdef HAVE_STDINT_H
#include <stdint.h>
#else
/* (Ronald) hacky... can't include stdint.h because it's not available
 * everywhere. however, a52dec wants uint8_t/uint32_t... how? */
#ifndef __uint8_t_defined
#define __uint8_t_defined
typedef guint8 uint8_t;
#endif

#ifndef __uint32_t_defined
#define __uint32_t_defined
typedef guint32 uint32_t;
#endif
/* grosj... but it works (tm) */
#endif /* HAVE_STDINT_H */


on solaris these types are defined in
#include <sys/types.h>

just typedef'ing to guint8/32 does not work as you've not included glib.h ;-)
Comment 1 Benjamin Otte (Company) 2004-02-03 02:38:24 UTC
2004-02-03  Benjamin Otte  <in7y118@public.uni-hamburg.de>
 
        * common/m4/ax_create_stdint_h.m4:
        * configure.ac:
          add AX_CREATE_STDINT_H to get correct type definitions for
a52dec in
          _stdint.h.
        * Makefile.am:
          remove generated _stdint.h in DISTCLEANFILES
        * ext/a52dec/gsta52dec.c:
          include _stdint.h for a52dec. (should fix #133064)