On 10 May, 06:51 pm, facundobati...@gmail.com wrote: >I'm analysing the code in twisted/spread/flavors.py and >twisted/spread/pb.py, trying to remove the __cmp__ and cmp() in >RemoteCache, RemoteCacheMethod and RemoteCacheObserver classes. > >They all have __cmp__ methods, but as there is not any test for these, >and it's not specified in the documentation, I don't know if the >comparison method is to compare by equality, or for these objects to >be orderable. > >Yes, I could replace __cmp__ with the full rich comparison battery, >but it's not worth it if the semantic of these objects will not use >it.
I doubt anyone remembers the rationale for these methods. We should err on the safe side and preserve the existing behavior. Jean-Paul _______________________________________________ Twisted-Python mailing list Twisted-Python@twistedmatrix.com http://twistedmatrix.com/cgi-bin/mailman/listinfo/twisted-python
