GeezerVison is showing mail.python.org — [Python-Dev] Re: [Python-checkins] CVS: python/dist/src/Libfilecmp.py,1.6,1.7

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

Christian Tismer [email protected]
Sun, 03 Dec 2000 22:12:15 +0200

Previous message: [Python-Dev] Re: [Python-checkins] CVS: python/dist/src/Lib filecmp.py,1.6,1.7
Next message: [Python-Dev] PEP 231, __findattr__()
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

Moshe Zadka wrote:
> 
> On Sun, 3 Dec 2000, Christian Tismer 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.
> 
> The fine chance was when we moved cmp.py->filecmp.py.
> Now it would just break backwards compatability.

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:[email protected]>
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

Previous message: [Python-Dev] Re: [Python-checkins] CVS: python/dist/src/Lib filecmp.py,1.6,1.7
Next message: [Python-Dev] PEP 231, __findattr__()
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]