Compilation on 64 bit platforms

[abligh]

In order to compile on a 64 bit platform, you need to apply the following patch to 0.9.1 (available under the wxWidgets license trivial though the patch is). It should not affect 32-bit compilation.

Code: Select all

diff -u -r src/manager.cpp src/manager.cpp
--- wxaui-0.9.1.orig/src/manager.cpp    2006-01-10 17:11:39.000000000 +0000
+++ wxaui-0.9.1/src/manager.cpp 2006-01-10 20:14:30.000000000 +0000
@@ -1160,7 +1160,7 @@
     if (pinfo.name.IsEmpty())
     {
         pinfo.name.Printf(wxT("%08x%08x%08x%08x"),
-             (unsigned int)pinfo.window, (unsigned int)time(NULL),
+             pinfo.window, (unsigned int)time(NULL),
              (unsigned int)clock(), m_panes.GetCount());
     }




[Ben]

Wow, sorry. I thought I had applied that one. I guess it slipped through my fingers.

Sorry about that. I'll get it in next time.

[tenmon]

Sorry, it shouldn't be:

Code: Select all

+             (unsigned long)pinfo.window, (unsigned int)time(NULL),

instead of

Code: Select all

+             pinfo.window, (unsigned int)time(NULL),

or

Code: Select all

+             (unsigned int)pinfo.window, (unsigned int)time(NULL),

I'm asking because (in case you didn't know), on most platforms

Code: Select all

sizeof(void*) == sizeof(long)

On 32 bits: sizeof(void*) == 32 bits
On 32 bits: sizeof(int) == 32 bits
On 32 bits: sizeof(long) == 32 bits

On 64 bits: sizeof(void*) == 64 bits
On 64 bits: sizeof(int) == 32 bits
On 64 bits: sizeof(long) == 64 bits

That's the most common, at least.

[Guest]

Yup, although technically, it needs to be:

Code: Select all

((unsigned long)pinfo.window) & 0xffffffff


This is the way I ended up patching it.

The pointer is simply used as a randomizing element for a unique window name, when one is not specified.
[/code]