[aquaria] Start of a 64bit patch

Slyf aldredev at gmail.com
Thu Jun 3 16:11:41 EDT 2010


You used "long" which will not work on Windows64 as long in windows64
(LLP64) is 32bits.  My solution allows for more portability in the
future.  On a side note, on windows64 int should be switched to "long
long" in my patch.  My patch should also be switched so that unix32
uses int instead of long to save on memory.

On Thu, Jun 3, 2010 at 3:57 PM, Pontos <daemonpontos at gmail.com> wrote:
>
> In core.cpp you should not define PATH_MAX but include the file limits.h, like i did on the patch i posted an hour ago.
> #include <limits.h>
>
> Also, your patch changes the same lines as mine regarding the use of int, but in the end the result is the same since i see that you made the type CrossSysInt to be "long" in the new header file you created:
>
> #if defined(_WIN32)
>    typedef int CrossSysInt;
> #elif defined(_WIN64)
>    typedef int CrossSysInt;
> #else
>    typedef long CrossSysInt;
> #endif
>
> Why making it int only for Windows anyway? And as it is now the code wont compile for that platform, right?
>
> Pontos
>
> On 6/3/2010 4:48 PM, Slyf wrote:
>>
>> Attached is the start of a patch for 64bit support.  For some reason it renders all black for me, however the audio works fine.  Hopefully this will be some help in fixing that issue.
>
> _______________________________________________
> aquaria mailing list
> aquaria at icculus.org
> http://icculus.org/mailman/listinfo/aquaria


More information about the aquaria mailing list