Re: [Python-checkins] CVS: python/dist/src/Objects classobject.c,2.82,2.83

I didn't have the heart to keep pestering Moshe, so I delayed a bit on the latest round (and then left town for a few days). As a result, I didn't get a chance to comment on this last patch. There is one problem, and some formatting nits... On Mon, 28 Feb 2000, Guido van Rossum wrote:
"Standard Guido Formatting" requires a space between the "if" and the "(". if, for, while, etc are not functions... they are language constructs.
I had suggested to Moshe to follow the logic in PySequence_Contains(), but he wanted to use PyObject_Cmp() instead. No biggy, but the above code has a bug: PyObject_Cmp() does *not* guarantee a value for cmp_res if it returns -1. Therefore, it is possible for cmp_res to be zero despite an error being returned from PyObject_Cmp. Thus, you get a false-positive hit. IMO, this section of code should read: cmp_res = PyObject_Compare(obj, member); Py_XDECREF(obj); if (cmp_res == 0) return 1; if (PyErr_Occurred()) return -1; The "ret" variable becomes unused and can be deleted. Oh! Just noted that "ret" is declared twice; one hiding the declaration of the other. With the above change and deletion of the inner "ret", then the hiding declaration problem is also fixed. Cheers, -g -- Greg Stein, http://www.lyra.org/

On Mon, 28 Feb 2000, Greg Stein wrote:
I can't do anything about you leaving town, but I do hope you're not feeling you're pestering me... I *do* want comments: much better then bugs creeping into Python.
"Standard Guido Formatting" requires a space between the "if" and the "(". if, for, while, etc are not functions... they are language constructs.
I won't argue formatting style, but it's simply hard to change habits. (My company keeps bugging me about that too, but I simply have trouble with the space bar<wink>)
Oops. I'll resynch with CVS and correct the bug. -- Moshe Zadka <mzadka@geocities.com>. INTERNET: Learn what you know. Share what you don't.

On Mon, 28 Feb 2000, Greg Stein wrote:
I can't do anything about you leaving town, but I do hope you're not feeling you're pestering me... I *do* want comments: much better then bugs creeping into Python.
"Standard Guido Formatting" requires a space between the "if" and the "(". if, for, while, etc are not functions... they are language constructs.
I won't argue formatting style, but it's simply hard to change habits. (My company keeps bugging me about that too, but I simply have trouble with the space bar<wink>)
Oops. I'll resynch with CVS and correct the bug. -- Moshe Zadka <mzadka@geocities.com>. INTERNET: Learn what you know. Share what you don't.
participants (2)
-
Greg Stein
-
Moshe Zadka