From 93662417e9253ed87eb94f0b7b51491bfcca3eed Mon Sep 17 00:00:00 2001
From: Guido van Rossum <guido@python.org>
Date: Sat, 19 Aug 2006 16:09:41 +0000
Subject: [PATCH] More has_key() fixes. The optparse fix is a fix to the
 previous fix, which broke has_option().

---
 Lib/logging/__init__.py | 6 +++---
 Lib/optparse.py         | 2 +-
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/Lib/logging/__init__.py b/Lib/logging/__init__.py
index c65d07fc6b0..1ef8f4405b6 100644
--- a/Lib/logging/__init__.py
+++ b/Lib/logging/__init__.py
@@ -806,7 +806,7 @@ class PlaceHolder:
         Add the specified logger as a child of this placeholder.
         """
         #if alogger not in self.loggers:
-        if not self.loggerMap.has_key(alogger):
+        if alogger not in self.loggerMap:
             #self.loggers.append(alogger)
             self.loggerMap[alogger] = None
 
@@ -863,7 +863,7 @@ class Manager:
         rv = None
         _acquireLock()
         try:
-            if self.loggerDict.has_key(name):
+            if name in self.loggerDict:
                 rv = self.loggerDict[name]
                 if isinstance(rv, PlaceHolder):
                     ph = rv
@@ -891,7 +891,7 @@ class Manager:
         rv = None
         while (i > 0) and not rv:
             substr = name[:i]
-            if not self.loggerDict.has_key(substr):
+            if substr not in self.loggerDict:
                 self.loggerDict[substr] = PlaceHolder(alogger)
             else:
                 obj = self.loggerDict[substr]
diff --git a/Lib/optparse.py b/Lib/optparse.py
index a02f79a0988..0972f74bcce 100644
--- a/Lib/optparse.py
+++ b/Lib/optparse.py
@@ -1040,7 +1040,7 @@ class OptionContainer:
 
     def has_option(self, opt_str):
         return (opt_str in self._short_opt or
-                opt_str) in self._long_opt
+                opt_str in self._long_opt)
 
     def remove_option(self, opt_str):
         option = self._short_opt.get(opt_str)