Skip to content

Commit

Permalink
ate: Sun, 04 Jul 2004 11:53:23 +0200
Browse files Browse the repository at this point in the history
From: Josselin Mouette
Subject: Re: sdl 64bit problem [PATCH]

Le sam, 03/07/2004 ? 18:20 +0200, Hans-Frieder Vogt a ?crit :
> Hi list,
>
> I found the problem that lead to the distorted sound in tuxracer on AMD64. It
> may have also been the cause for the other reported sound problems with SDL.
> The problem is not in SDL, but in SDL-mixer.
> There, music files (*.it, and probably others as well) are incorrectly read on
> all 64 bit architectures but Alpha, due to an incorrect data type definition.
> The attached patch should solve the problem for all 64 bit architectures on
> Linux, since at least cpp defines _LP64 and __LP64__ for them.
> _LP64 indicates that long ints and pointers are 64 bits, but integers are 32
> bits.
> I am not sure whether the usage of _LP64 is also standard for the commercial
> compilers, and for other Unixes, but at least Solaris 9 defines _LP64 in the
> 64 bit environment as well.
>
> Josselin,
> could you include this patch into your next debian packages?

Certainly. Sam, could you please include it in the CVS as well?
  • Loading branch information
slouken committed Jul 18, 2004
1 parent 144bf49 commit 13578d3
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion mikmod/mikmod.h
Expand Up @@ -85,7 +85,7 @@ typedef char CHAR;

/*@DOES_NOT_HAVE_SIGNED@*/

#if defined(__alpha)
#if defined(__alpha) || defined(_LP64)
/* 64 bit architectures */

typedef signed char SBYTE; /* 1 byte, signed */
Expand Down
2 changes: 1 addition & 1 deletion mikmod/mikmod_internals.h
Expand Up @@ -69,7 +69,7 @@ extern "C" {
/*========== More type definitions */

/* SLONGLONG: 64bit, signed */
#if defined(__alpha)
#if defined(__alpha) || defined(_LP64)
typedef long SLONGLONG;
#define NATIVE_64BIT_INT
#elif defined(__WATCOMC__)
Expand Down

0 comments on commit 13578d3

Please sign in to comment.