[Patches] [ python-Patches-1141428 ] more __contains__ tests

SourceForge.net noreply at sourceforge.net
Fri Feb 18 01:46:17 CET 2005


Patches item #1141428, was opened at 2005-02-17 19:42
Message generated for change (Settings changed) made by jimjjewett
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1141428&group_id=5470

>Category: Tests
>Group: Python 2.4
Status: Open
Resolution: None
>Priority: 4
Submitted By: Jim Jewett (jimjjewett)
Assigned to: Nobody/Anonymous (nobody)
Summary: more __contains__ tests

Initial Comment:
Recently, there was a change to speed up 
__contains__.  

x in (1, 2, 3) would be compiled as 
x in frozenset(1, 2, 3).

This was backed out, because frozensets raise a 
typeerror when asked about unhashable objects.  
These objects *can't* be in the set, but there was 
concern about supporting objects in which a mutable 
and an immutable compare equal.

So long as that is a requirement, it should be tested.  

The new test for sequences ensures that they will use 
__eq__ (and not rely on __hash__) in the future.  

The change to the rich compare tests is just a 
comment, pointing out that dicts can rely on 
__hash__ as well as __eq__.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1141428&group_id=5470


More information about the Patches mailing list