Mercurial > cpython-lru_cache-weakref
diff Lib/optparse.py @ 39575:63237fcefd99
Get rid of dict.has_key(). Boy this has a lot of repercussions!
Not all code has been fixed yet; this is just a checkpoint...
The C API still has PyDict_HasKey() and _HasKeyString(); not sure
if I want to change those just yet.
| author | Guido van Rossum <guido@python.org> |
|---|---|
| date | Fri, 18 Aug 2006 22:13:04 +0000 |
| parents | 654c380cf8b9 |
| children | d240ce5696f9 |
line wrap: on
line diff
--- a/Lib/optparse.py Fri Aug 18 16:29:54 2006 +0000 +++ b/Lib/optparse.py Fri Aug 18 22:13:04 2006 +0000 @@ -602,7 +602,7 @@ def _set_attrs(self, attrs): for attr in self.ATTRS: - if attrs.has_key(attr): + if attr in attrs: setattr(self, attr, attrs[attr]) del attrs[attr] else: @@ -854,7 +854,7 @@ are silently ignored. """ for attr in dir(self): - if dict.has_key(attr): + if attr in dict: dval = dict[attr] if dval is not None: setattr(self, attr, dval) @@ -974,10 +974,10 @@ def _check_conflict(self, option): conflict_opts = [] for opt in option._short_opts: - if self._short_opt.has_key(opt): + if opt in self._short_opt: conflict_opts.append((opt, self._short_opt[opt])) for opt in option._long_opts: - if self._long_opt.has_key(opt): + if opt in self._long_opt: conflict_opts.append((opt, self._long_opt[opt])) if conflict_opts: @@ -1023,7 +1023,7 @@ if option.dest is not None: # option has a dest, we need a default if option.default is not NO_DEFAULT: self.defaults[option.dest] = option.default - elif not self.defaults.has_key(option.dest): + elif option.dest not in self.defaults: self.defaults[option.dest] = None return option @@ -1039,8 +1039,8 @@ self._long_opt.get(opt_str)) def has_option(self, opt_str): - return (self._short_opt.has_key(opt_str) or - self._long_opt.has_key(opt_str)) + return (opt_str in self._short_opt or + opt_str) in self._long_opt def remove_option(self, opt_str): option = self._short_opt.get(opt_str) @@ -1658,7 +1658,7 @@ 'words', raise BadOptionError. """ # Is there an exact match? - if wordmap.has_key(s): + if s in wordmap: return s else: # Isolate all words with s as a prefix.