RE: [C++-sig] Patches for pyste regarding exception specification s
...
Thanks a lot Gottfried, your patch looks very clean! I will commit it as soon as Prabhu and I have fixed the "Order Bug". Good luck!
Just one thing I didn't understand, could you explain this change?
- def SetCurrent(self, current): + def SetCurrent(self, current, missing = None): + if missing is not None: + pass +
This is one of the "fixes" I mentioned in my post. Originally I planned to print a message or throw an exception when missing wouldn't be None because I didn't know what to make of that parameter. Without it I got a 'wrong number of parameters' exception but I somehow got distracted and forgot about it when I came back and didn't get any more of those exceptions. I guess that this parameter should really be called "unused" because it needs to be there only for compatibility to MultipleCodeUnit.SetCurrent(), right?
Thanks again!
my pleasure
Nicodemus.
Cheers, Gottfried
Gottfried.Ganssauge@HAUFE.DE wrote:
...
Thanks a lot Gottfried, your patch looks very clean! I will commit it as soon as Prabhu and I have fixed the "Order Bug". Good luck!
I Gottfried! I implemented your patch in the current CVS. I left out the ability of exporting exceptions that you proposed in the "Wrapper for exception translation" thread, because it seems that this is still in progress, but as soon as it is ready I will be glad to add support for it in Pyste. Thanks again! Nicodemus.
participants (2)
-
Gottfried.Ganssauge@HAUFE.DE -
Nicodemus