[prev in list] [next in list] [prev in thread] [next in thread] 

List:       pykde
Subject:    Re: [PyQt] sip: extend exception support
From:       Mathias.Born () gmx ! de
Date:       2012-09-19 21:17:56
Message-ID: 179116387.20120919231756 () gmx ! de
[Download RAW message or body]

On 19.09.2012, 11:07:14 Phil Thompson wrote:
> On Mon, 17 Sep 2012 22:25:59 +0200, Mathias.Born@gmx.de wrote:
>> On 17.09.2012, 22:14:49 Phil Thompson wrote:
>>> On Mon, 27 Aug 2012 12:10:23 -0700, Matt Newell <newellm@blur.com>
> wrote:
>>>>> 
>>>>> The difference is the access to the (not necessarily present)
> objects.
>>>>> How
>>>>> are you getting these?
>>>>> 
>>>>> Phil
>>>> 
>>>> Here's the patch against a quite old hg checkout.
>> 
>>> After several more iterations (ok, rewrites) the implementation in the
>>> current snapshot should be Ok - and work with %Imported modules.
>> 
>>> Phil
>> 
>> I'm pleased to report that this works fine for me. Some comments:
>> 
>> In the docs about the "%DefaultVirtualErrorHandler" directive, there is
> a
>> typo
>> 
>> %DefaultVirtualErrorHandle my_handler
>> 
>> which should read
>> 
>> %DefaultVirtualErrorHandler my_handler
>> 
>> 
>> In addition, the function type is not
>> 
>> void my_handler(sip_gilstate_t, PyObject *);
>> 
>> but instead
>> 
>> void my_handler(sip_gilstate_t, sipSimpleWrapper*);

> You need to use the default branch (or a snapshot) - I changed it all
> again.

Ok, but that's not working properly anymore.

sip.h declares

typedef void (*sipVirtErrorHandlerFunc)(struct _sipSimpleWrapper *);

but the generated code containes error handlers that look like

void sipVEH_ltse_app_defaultVirtualErrorHandler(sip_gilstate_t, PyObject *)

And inside other functions they are referred to as, for example

extern void sipVEH_ltse_app_defaultVirtualErrorHandler(sip_gilstate_t, PyObject *);

Best Regards,
Mathias Born


_______________________________________________
PyQt mailing list    PyQt@riverbankcomputing.com
http://www.riverbankcomputing.com/mailman/listinfo/pyqt
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic