Message49034
Logged In: YES
user_id=1038590
Closest yet, but I don't think we're quite there yet.
The 'except_st = NULL' part should be on the line
immediately after except_st is inserted into the inner
sequence. As soon as that insertion happens, we want to
ensure that the statement is only freed once. What's
currently there will technically work, but doesn't really
have the right semantics.
More importantly, the "body = handlers = orelse = NULL" part
should only be executed if the call to TryExcept *succeeds*,
not if it fails.
Neil's right - we seriously need to find a better way to
handle this memory management or it will be a source of
significant pain (well, more pain than you've already
experienced ;) |
|
Date |
User |
Action |
Args |
2007-08-23 15:44:33 | admin | link | issue1355913 messages |
2007-08-23 15:44:33 | admin | create | |
|