Re: [patch] VOIDP2INT, INT2VOIDP and pointer size

Johannes Hofmann <[email protected]>
Newsgroups gmane.comp.web.dillo.devel
Message-ID <[email protected]>
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
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.