[PATCH][win32] problems compiling bzrlib/_dirstate_helpers_c.pyx with MSVC

Lukáš Lalinský lalinsky at gmail.com
Tue Jul 31 10:16:40 BST 2007


On Ut, 2007-07-31 at 12:13 +0300, Alexander Belchenko wrote: 
> I don't know is it bug in Pyrex or in our usage of Pyrex, but I repeat
> one more time:
> 
> 	Pyrex substitute intptr_t with int in
> 	generated C-code, so actually nor stdint.h nor intptr_t
> 	is not used at all.
> 
> intptr_t is not present in generated C-code. So, it's already
> breaks compatibility with 64-bit platform.

If the ctypedef statement appears in a 'cdef extern from' section it
will not be replaced. This happens only if it's moved to the top-level
block.

For example my compiled _dirstate_helpers_c.c contains:

  __pyx_r = ((((intptr_t )__pyx_v_ptr) & ((sizeof(int )) - 1)) == 0);

I think the attached patch is a better solution.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: bzr-pyrex-msvc.diff
Type: text/x-patch
Size: 963 bytes
Desc: not available
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20070731/aeb6a49a/attachment.bin 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Toto je =?ISO-8859-1?Q?digit=E1lne?=
	=?ISO-8859-1?Q?_podp=EDsan=E1?= =?UTF-8?Q?_=C4=8Das=C5=A5?=
	=?ISO-8859-1?Q?_spr=E1vy?=
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20070731/aeb6a49a/attachment.pgp 


More information about the bazaar mailing list