Hi Christoph (and markusstoll)
The reason this isn't compiling is because of a change made by markusstoll on 18 April to Utilities/Adaptors/Adaptor/shmem.c .With his changes, shown below, WOShmem_base_address is type (void *) and on line 78 offset is cast to (void *). The compiler is rightfully complaining that you are adding a pointer address to a pointer address which you would never want to do. I changed line 78 to remove the cast to (void *) of offset and it now compiles. I haven't tested it though so I am not sure of the ramifications of my change or other changes made by markusstoll. (In markusstoll's defence, the original code was probably also wrong, it was just the compiler couldn't pick up the error).
The following is the diff shown on github between before and after markusstoll's change.
| 69
|
|
-static intptr_t WOShmem_base_address = -1;
|
|
|
69
|
+static void * WOShmem_base_address = -1;
|
|
70
|
70
|
|
|
71
|
71
|
/*
|
|
72
|
72
|
* The total size of the mapped memory.
|
|
|
@@ -75,8 +75,8 @@ static unsigned int WOShmem_size = 0;
|
|
75
|
75
|
|
|
76
|
76
|
static WA_recursiveLock WOShmem_mutex;
|
|
77
|
77
|
|
|
78
|
|
-#define offset_to_addr(offset) ((void *)(WOShmem_base_address + (intptr_t)offset))
|
|
79
|
|
-#define addr_to_offset(addr) ((intptr_t)addr - WOShmem_base_address)
|
|
|
78
|
+#define offset_to_addr(offset) ((void *)(WOShmem_base_address + (void *)offset))
|
|
|
79
|
Do not post admin requests to the list. They will be ignored.
Webobjects-dev mailing list (email@hidden)
Help/Unsubscribe/Update your Subscription:
This email sent to email@hidden
|
|