This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author Marko.Kohtala
Recipients Marko.Kohtala, eric.araujo, ghaering
Date 2010-09-12.08:57:19
SpamBayes Score 7.8145916e-08
Marked as misclassified No
Message-id <1284281842.64.0.959923298138.issue9750@psf.upfronthosting.co.za>
In-reply-to
Content
Thank you for the review.

I have very limited time to use on this. So even when I'd like to make everything easy for you, have the time you give to python be as productive as possible, I can not.

But I'll respond to your comments on the patch.

a) I added the quotes to every identifier based on a comment in sqlite documentation "SQLite adds new keywords from time to time when it takes on new features. So to prevent your code from being broken by future enhancements, you should normally quote any identifier that is an English language word, even if you do not have to." While fixing one place, I fixed it to follow this recommendation in other places as well.

b) I added quotes using backslashes because it was consistent. The table name was already quoted like that. I agree it could be clearer.

c) I know. I only tried to make minimal changes targeted only to the issues at hand. I did not want to hide the fixes in middlde of changes to style.
History
Date User Action Args
2010-09-12 08:57:22Marko.Kohtalasetrecipients: + Marko.Kohtala, ghaering, eric.araujo
2010-09-12 08:57:22Marko.Kohtalasetmessageid: <1284281842.64.0.959923298138.issue9750@psf.upfronthosting.co.za>
2010-09-12 08:57:20Marko.Kohtalalinkissue9750 messages
2010-09-12 08:57:19Marko.Kohtalacreate