[Dillo-dev] [patch] VOIDP2INT, INT2VOIDP and pointer size

On Tue, Jun 12, 2012 at 11:25:33PM +0400, 123 wrote:

Now (long) is used for converting (void *) to integer. This patch replaces (long) with intptr_t.

I don't think this fixes the real problem. When VOIDP2INT and INT2VOIDP macros are used, it is possible that converted type won't fit into intptr_t. It is better to rewrite code that uses these macros in portable way and remove these macros.

Agreed, so let's go for the real fix - step by step. Even better would be if we could come up with a case where the current scheme causes a real problem - not sure if this is possible, some compiler warnings maybe? Cheers, Johannes