Re: [Pypgsql-users] Savepoint or subtransaction support

[Markus S. <ma...@bl...>]

Hi,

Clarence Gardner wrote:
> Actually, PostgreSQL is following the SQL standard in that respect. 
> Given that, I think your patch has an error when it's searching for the 
> savepoint name to rollback to. In case the same name is currently active 
> more than once, you want to search from the end of your list of 
> savepoint names rather than from the beginning (since savepoint() 
> appends to the list).

Good catch! I didn't think about that. Can we use rindex() or do we have 
to support python <= 2.3? What's the best way to do it for python 2.3? 
Does somebody know a more elegant solution than:

a = [...]
a.reverse()
idx = len(a) - a.index(..) - 1
a.reverse()

>> (I've tried to upload the page to the 'patches' page on SourceForge, 
>> but didn't succeed, I've only added a useless comment, sorry. Please 
>> bear with a SF-first-timer.)
>>
> I had what may be the same problem. Even if you set the file to upload, 
> the server ignores it unless you check the checkbox labeled "Upload 
> file" (or something).

Hm... I didn't even get to an 'upload file' dialog. I thought that would 
show up when confirming your comment. But despite all the other 
'confirm' buttons you have to click when subscribing, your comment gets 
added instantly.

Anyway, I now suppose I have to be a member of the project to add 
patches? Or how do I do that?

Regards

Markus