From: Benjamin Peterson <benjamin@python.org>
Date: Tue, 1 Apr 2014 18:20:56 +0000 (-0400)
Subject: simplify check, since now there are only new-style classes
X-Git-Tag: v3.4.1rc1~142
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=5dd3caed2be6da5df934c7c3bedf0bd9ba2d4bf6;p=python

simplify check, since now there are only new-style classes
---

diff --git a/Lib/urllib/request.py b/Lib/urllib/request.py
index 5995cbe24f..65a3aeb72f 100644
--- a/Lib/urllib/request.py
+++ b/Lib/urllib/request.py
@@ -511,9 +511,6 @@ def build_opener(*handlers):
     If any of the handlers passed as arguments are subclasses of the
     default handlers, the default handlers will not be used.
     """
-    def isclass(obj):
-        return isinstance(obj, type) or hasattr(obj, "__bases__")
-
     opener = OpenerDirector()
     default_classes = [ProxyHandler, UnknownHandler, HTTPHandler,
                        HTTPDefaultErrorHandler, HTTPRedirectHandler,
@@ -524,7 +521,7 @@ def build_opener(*handlers):
     skip = set()
     for klass in default_classes:
         for check in handlers:
-            if isclass(check):
+            if instance(check, type):
                 if issubclass(check, klass):
                     skip.add(klass)
             elif isinstance(check, klass):
@@ -536,7 +533,7 @@ def build_opener(*handlers):
         opener.add_handler(klass())
 
     for h in handlers:
-        if isclass(h):
+        if isinstance(h, type):
             h = h()
         opener.add_handler(h)
     return opener