On mer, 2009-01-14 at 12:24 -0800, [email protected] wrote:
> Wow, maybe I am an expert after all... :P
> 
> Index: source/common/launch.h
> ===================================================================
> --- source/common/launch.h      (revision 594)
> +++ source/common/launch.h      (working copy)
> @@ -79,16 +79,17 @@
>   */
>  /*typedef _typeobject;*/
>  typedef struct _object {
> -    int ob_refcnt;
> +    /*int ob_refcnt; */
> +    ssize_t ob_refcnt;
>      struct _typeobject *ob_type;
>  } PyObject;
>  typedef void (*destructor)(PyObject *);
>  typedef struct _typeobject {
> -    int ob_refcnt;
> +    ssize_t ob_refcnt;
>      struct _typeobject *ob_type;
> -    int ob_size;
> +    ssize_t ob_size;
>      char *tp_name; /* For printing */
> -    int tp_basicsize, tp_itemsize; /* For allocation */
> +    ssize_t tp_basicsize, tp_itemsize; /* For allocation */
>      destructor tp_dealloc;
>      /* ignore the rest.... */
>  } PyTypeObject;
> 

Wow, interesting... I didn't even know that those lines were there...
before applying this patch (which btw doesn't compile on Windows which
doesn't define "ssize_t") I would like to double-check why we can't
simply include "Python.h" instead of having this code here. Probably the
idea was to be able to compile the bootloader without having Python
includes around; but the includes are shipped with Python even on
Windows, so I don't see the big deal.

I'll investigate. Thanks!
-- 
Giovanni Bajo
Develer S.r.l.
http://www.develer.com



--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"PyInstaller" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to 
[email protected]
For more options, visit this group at 
http://groups.google.com/group/PyInstaller?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to