diff --git a/Lib/cgi.py b/Lib/cgi.py index 818567e5fdc..8760f960549 100755 --- a/Lib/cgi.py +++ b/Lib/cgi.py @@ -607,31 +607,27 @@ def keys(self): """Dictionary style keys() method.""" if self.list is None: raise TypeError, "not indexable" - keys = [] - for item in self.list: - if item.name not in keys: keys.append(item.name) - return keys + return list(set(item.name for item in self.list)) def has_key(self, key): """Dictionary style has_key() method.""" if self.list is None: raise TypeError, "not indexable" - for item in self.list: - if item.name == key: return True - return False + return any(item.name == key for item in self.list) def __contains__(self, key): """Dictionary style __contains__ method.""" if self.list is None: raise TypeError, "not indexable" - for item in self.list: - if item.name == key: return True - return False + return any(item.name == key for item in self.list) def __len__(self): """Dictionary style len(x) support.""" return len(self.keys()) + def __nonzero__(self): + return bool(self.list) + def read_urlencoded(self): """Internal: read data in query string format.""" qs = self.fp.read(self.length) diff --git a/Misc/NEWS b/Misc/NEWS index 4503875f853..2dadbbfeca4 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -260,6 +260,8 @@ Core and builtins Library ------- +- Patch #1541463: optimize performance of cgi.FieldStorage operations. + - Decimal is fully updated to the latest Decimal Specification (v1.66). - Bug #1153: repr.repr() now doesn't require set and dictionary items