Re: [Python-checkins] CVS: python/dist/src/Lib filecmp.py,1.6,1.7

Moshe Zadka wrote:
...
Ugh! Wouldn't that be a fine chance to rename the cmp function in this module? Overriding a built-in is really not nice to have in a library. ciao - chris -- Christian Tismer :^) <mailto:tismer@tismer.com> Mission Impossible 5oftware : Have a break! Take a ride on Python's Kaunstr. 26 : *Starship* http://starship.python.net 14163 Berlin : PGP key -> http://wwwkeys.pgp.net PGP Fingerprint E182 71C7 1A9D 66E9 9D15 D3CC D4D7 93E2 1FAE F6DF where do you want to jump today? http://www.stackless.com

On Sun, 3 Dec 2000, Christian Tismer wrote:
The fine chance was when we moved cmp.py->filecmp.py. Now it would just break backwards compatability. -- Moshe Zadka <moshez@zadka.site.co.il> -- 95855124 http://moshez.org

Moshe Zadka wrote:
Yes, I see. cmp belongs to the module's interface. Maybe it could be renamed anyway, and be assigned to cmp at the very end of the file, but not using cmp anywhere in the code. My first reaction on reading the patch was "juck!" since I didn't know this module. python-dev/null - ly y'rs - chris -- Christian Tismer :^) <mailto:tismer@tismer.com> Mission Impossible 5oftware : Have a break! Take a ride on Python's Kaunstr. 26 : *Starship* http://starship.python.net 14163 Berlin : PGP key -> http://wwwkeys.pgp.net PGP Fingerprint E182 71C7 1A9D 66E9 9D15 D3CC D4D7 93E2 1FAE F6DF where do you want to jump today? http://www.stackless.com

On Sun, 3 Dec 2000, Christian Tismer wrote:
The fine chance was when we moved cmp.py->filecmp.py. Now it would just break backwards compatability. -- Moshe Zadka <moshez@zadka.site.co.il> -- 95855124 http://moshez.org

Moshe Zadka wrote:
Yes, I see. cmp belongs to the module's interface. Maybe it could be renamed anyway, and be assigned to cmp at the very end of the file, but not using cmp anywhere in the code. My first reaction on reading the patch was "juck!" since I didn't know this module. python-dev/null - ly y'rs - chris -- Christian Tismer :^) <mailto:tismer@tismer.com> Mission Impossible 5oftware : Have a break! Take a ride on Python's Kaunstr. 26 : *Starship* http://starship.python.net 14163 Berlin : PGP key -> http://wwwkeys.pgp.net PGP Fingerprint E182 71C7 1A9D 66E9 9D15 D3CC D4D7 93E2 1FAE F6DF where do you want to jump today? http://www.stackless.com
participants (2)
-
Christian Tismer
-
Moshe Zadka