Swap the key check and use casefold in the test, as suggested in review
This commit is contained in:
1 parent
8780c614af
commit
dce315d944
2 files changed
+2
-2
No files matched your search
@@ -4279,7 +4279,7 @@ def all_unique(iterable, key=None):
|
|||||||
seenset_add = seenset.add
|
seenset_add = seenset.add
|
||||||
seenlist = []
|
seenlist = []
|
||||||
seenlist_add = seenlist.append
|
seenlist_add = seenlist.append
|
||||||
for element in map(key, iterable) if key is not None else iterable:
|
for element in iterable if key is None else map(key, iterable):
|
||||||
try:
|
try:
|
||||||
if element in seenset:
|
if element in seenset:
|
||||||
return False
|
return False
|
||||||
|
|||||||
+1
-1
@@ -5160,7 +5160,7 @@ class AllUniqueTests(TestCase):
|
|||||||
return False
|
return False
|
||||||
|
|
||||||
def __call__(self, x):
|
def __call__(self, x):
|
||||||
return x.lower()
|
return x.casefold()
|
||||||
|
|
||||||
key = CaseFold()
|
key = CaseFold()
|
||||||
self.assertFalse(key) # Sanity: the key itself is falsey
|
self.assertFalse(key) # Sanity: the key itself is falsey
|
||||||
|
|||||||
Reference in new issue
Block a user