Message 66007 - Python tracker

➜

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	gvanrossum
Recipients	belopolsky, georg.brandl, gvanrossum, rhettinger
Date	2008-04-30.16:48:49
SpamBayes Score	0.12589812
Marked as misclassified	No
Message-id	<[email protected]>
In-reply-to

Content
+1 on this. I have a few nits about the code: Line 1083: "%.200s object is not an iterator", it->ob_type->tp_name); Line is too long. Line 1088: if (res == NULL) { How about if (res != NULL) return res; ? Line 1089: if (def) { if (def != NULL) { Line 1093: PyErr_Clear(); I would only call this if PyErr_Occurred() returns true.

+1 on this. I have a few nits about the code:

Line 1083: "%.200s object is not an iterator", it->ob_type->tp_name);
Line is too long.

Line 1088: if (res == NULL) {
How about

  if (res != NULL)
    return res;

?

Line 1089: if (def) {
if (def != NULL) {

Line 1093: PyErr_Clear();
I would only call this if PyErr_Occurred() returns true.

History
Date	User	Action	Args
2008-04-30 16:48:51	gvanrossum	set	spambayes_score: 0.125898 -> 0.12589812 recipients: + gvanrossum, georg.brandl, rhettinger, belopolsky
2008-04-30 16:48:51	gvanrossum	set	spambayes_score: 0.125898 -> 0.125898 messageid: <[email protected]>
2008-04-30 16:48:50	gvanrossum	link	issue2719 messages
2008-04-30 16:48:49	gvanrossum	create