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

Lukáš Lalinský lalinsky at gmail.com
Wed Aug 1 11:49:12 BST 2007


On St, 2007-08-01 at 13:39 +0300, Alexander Belchenko wrote:
> Lukáš Lalinský пишет:
> > 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);
> 
> Lukáš, what version of Pyrex you are using?

0.9.3.1 on Windows, 0.9.4.1 on Linux. But I've just noticed that the
newer version on Linux indeed replaces intptr_t with int. I personally
think this behavior is wrong, but I'm really not sure if it's a
regression or a new feature.

-------------- 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/20070801/787753c6/attachment.pgp 


More information about the bazaar mailing list