]> git.ipfire.org Git - thirdparty/Python/cpython.git/commitdiff
bpo-44712: Replace "type(literal)" with corresponding builtin types (GH-27294)
authorSerhiy Storchaka <storchaka@gmail.com>
Sun, 8 May 2022 14:10:11 +0000 (17:10 +0300)
committerGitHub <noreply@github.com>
Sun, 8 May 2022 14:10:11 +0000 (17:10 +0300)
I suppose it is a remnants of very old code written when str, int, list, dict, etc
were functions and not classes.

19 files changed:
Lib/cgitb.py
Lib/csv.py
Lib/curses/ascii.py
Lib/distutils/command/register.py
Lib/ftplib.py
Lib/getopt.py
Lib/idlelib/browser.py
Lib/lib2to3/pgen2/token.py
Lib/plistlib.py
Lib/pty.py
Lib/pydoc.py
Lib/sunau.py
Lib/test/test_copyreg.py
Lib/test/test_descr.py
Lib/test/test_pprint.py
Lib/test/test_timeout.py
Lib/xmlrpc/client.py
Mac/BuildScript/build-installer.py
Tools/scripts/mailerdaemon.py

index 8ce0e833a989aa6a9f7f886763c26bd4dfe482f2..f6b97f25c59de9871684c2757f9f0e792a7bbc81 100644 (file)
@@ -74,7 +74,7 @@ def lookup(name, frame, locals):
         return 'global', frame.f_globals[name]
     if '__builtins__' in frame.f_globals:
         builtins = frame.f_globals['__builtins__']
-        if type(builtins) is type({}):
+        if isinstance(builtins, dict):
             if name in builtins:
                 return 'builtin', builtins[name]
         else:
index bb3ee269ae7931f700efad706cbc3ed40c8be071..a0782705d5af1af1071cb6c0f70ce30723d504fb 100644 (file)
@@ -428,7 +428,7 @@ class Sniffer:
         # on whether it's a header
         hasHeader = 0
         for col, colType in columnTypes.items():
-            if type(colType) == type(0): # it's a length
+            if isinstance(colType, int): # it's a length
                 if len(header[col]) != colType:
                     hasHeader += 1
                 else:
index 5b243be68111fcfc3b3917d03d45e80fb7397186..95acff33925ed7a22224b54bc63ec5bbfdd2e949 100644 (file)
@@ -46,7 +46,7 @@ controlnames = [
 ]
 
 def _ctoi(c):
-    if type(c) == type(""):
+    if isinstance(c, str):
         return ord(c)
     else:
         return c
@@ -69,19 +69,19 @@ def isctrl(c): return 0 <= _ctoi(c) < 32
 def ismeta(c): return _ctoi(c) > 127
 
 def ascii(c):
-    if type(c) == type(""):
+    if isinstance(c, str):
         return chr(_ctoi(c) & 0x7f)
     else:
         return _ctoi(c) & 0x7f
 
 def ctrl(c):
-    if type(c) == type(""):
+    if isinstance(c, str):
         return chr(_ctoi(c) & 0x1f)
     else:
         return _ctoi(c) & 0x1f
 
 def alt(c):
-    if type(c) == type(""):
+    if isinstance(c, str):
         return chr(_ctoi(c) | 0x80)
     else:
         return _ctoi(c) | 0x80
index 0fac94e9e54905688d0e359fc5a9b96b703afab5..170f5497141c9dcb4e7b57c3cfdc54aa6b532d76 100644 (file)
@@ -260,7 +260,7 @@ Your selection [default 1]: ''', log.INFO)
         body = io.StringIO()
         for key, value in data.items():
             # handle multiple entries for the same name
-            if type(value) not in (type([]), type( () )):
+            if not isinstance(value, (list, tuple)):
                 value = [value]
             for value in value:
                 value = str(value)
index 7c5a50715f6dc691443ba89bfef07bbbea7fdc1d..dc9a8afbd8d247942b63bc01cf1816b893590a54 100644 (file)
@@ -561,7 +561,7 @@ class FTP:
         LIST command.  (This *should* only be used for a pathname.)'''
         cmd = 'LIST'
         func = None
-        if args[-1:] and type(args[-1]) != type(''):
+        if args[-1:] and not isinstance(args[-1], str):
             args, func = args[:-1], args[-1]
         for arg in args:
             if arg:
index 9d4cab1bac360dda9d49dd8eef258f23dc252a2b..5419d77f5d774e3d85bfe54f6277af88a0774cf8 100644 (file)
@@ -81,7 +81,7 @@ def getopt(args, shortopts, longopts = []):
     """
 
     opts = []
-    if type(longopts) == type(""):
+    if isinstance(longopts, str):
         longopts = [longopts]
     else:
         longopts = list(longopts)
index 3c3a53a6599a79282c0141e157712ae71cbc8c44..4da2d4071ac191871efb495df5aff3b8e7be3cb1 100644 (file)
@@ -43,7 +43,7 @@ def transform_children(child_dict, modname=None):
                 # If obj.name != key, it has already been suffixed.
                 supers = []
                 for sup in obj.super:
-                    if type(sup) is type(''):
+                    if isinstance(sup, str):
                         sname = sup
                     else:
                         sname = sup.name
index 5f6612f5b30681dab79e4f71800850202c43a9aa..2a55138e48237b3b57f7077acf718495a15f6353 100755 (executable)
@@ -72,7 +72,7 @@ NT_OFFSET = 256
 
 tok_name = {}
 for _name, _value in list(globals().items()):
-    if type(_value) is type(0):
+    if isinstance(_value, int):
         tok_name[_value] = _name
 
 
index 4862355b2252c98a11c8af70014b09887959cd85..d03c75dfab9756a3213d7a83a9c0654482ed4c35 100644 (file)
@@ -199,7 +199,7 @@ class _PlistParser:
 
     def add_object(self, value):
         if self.current_key is not None:
-            if not isinstance(self.stack[-1], type({})):
+            if not isinstance(self.stack[-1], dict):
                 raise ValueError("unexpected element at line %d" %
                                  self.parser.CurrentLineNumber)
             self.stack[-1][self.current_key] = value
@@ -208,7 +208,7 @@ class _PlistParser:
             # this is the root object
             self.root = value
         else:
-            if not isinstance(self.stack[-1], type([])):
+            if not isinstance(self.stack[-1], list):
                 raise ValueError("unexpected element at line %d" %
                                  self.parser.CurrentLineNumber)
             self.stack[-1].append(value)
@@ -232,7 +232,7 @@ class _PlistParser:
         self.stack.pop()
 
     def end_key(self):
-        if self.current_key or not isinstance(self.stack[-1], type({})):
+        if self.current_key or not isinstance(self.stack[-1], dict):
             raise ValueError("unexpected key at line %d" %
                              self.parser.CurrentLineNumber)
         self.current_key = self.get_data()
index 8d8ce40df541c1872b52ae6ea069e0e975bd8a6b..03073f07c92c052d74df3ac56e82c453fb2c0c12 100644 (file)
@@ -162,7 +162,7 @@ def _copy(master_fd, master_read=_read, stdin_read=_read):
 
 def spawn(argv, master_read=_read, stdin_read=_read):
     """Create a spawned process."""
-    if type(argv) == type(''):
+    if isinstance(argv, str):
         argv = (argv,)
     sys.audit('pty.spawn', argv)
 
index 297ff967a2ce9cb0ad3c33b69c9ea7173eb641b5..cec9ac89f1cc8d9108bc482f1a14348700e9c063 100755 (executable)
@@ -727,7 +727,7 @@ class HTMLDoc(Doc):
         """Produce HTML for a class tree as given by inspect.getclasstree()."""
         result = ''
         for entry in tree:
-            if type(entry) is type(()):
+            if isinstance(entry, tuple):
                 c, bases = entry
                 result = result + '<dt class="heading-text">'
                 result = result + self.classlink(c, modname)
@@ -737,7 +737,7 @@ class HTMLDoc(Doc):
                         parents.append(self.classlink(base, modname))
                     result = result + '(' + ', '.join(parents) + ')'
                 result = result + '\n</dt>'
-            elif type(entry) is type([]):
+            elif isinstance(entry, list):
                 result = result + '<dd>\n%s</dd>\n' % self.formattree(
                     entry, modname, c)
         return '<dl>\n%s</dl>\n' % result
@@ -1190,14 +1190,14 @@ class TextDoc(Doc):
         """Render in text a class tree as returned by inspect.getclasstree()."""
         result = ''
         for entry in tree:
-            if type(entry) is type(()):
+            if isinstance(entry, tuple):
                 c, bases = entry
                 result = result + prefix + classname(c, modname)
                 if bases and bases != (parent,):
                     parents = (classname(c, modname) for c in bases)
                     result = result + '(%s)' % ', '.join(parents)
                 result = result + '\n'
-            elif type(entry) is type([]):
+            elif isinstance(entry, list):
                 result = result + self.formattree(
                     entry, modname, c, prefix + '    ')
         return result
@@ -2044,7 +2044,7 @@ has the same effect as typing a particular string at the help> prompt.
             return self.input.readline()
 
     def help(self, request):
-        if type(request) is type(''):
+        if isinstance(request, str):
             request = request.strip()
             if request == 'keywords': self.listkeywords()
             elif request == 'symbols': self.listsymbols()
@@ -2129,7 +2129,7 @@ module "pydoc_data.topics" could not be found.
         if not target:
             self.output.write('no documentation found for %s\n' % repr(topic))
             return
-        if type(target) is type(''):
+        if isinstance(target, str):
             return self.showtopic(target, more_xrefs)
 
         label, xrefs = target
index 94c42f1582c021db4b8862d6efe2e13af35948ed..c6caab959abd0cd428b77d3d9b4f1092d50bb6e5 100644 (file)
@@ -160,7 +160,7 @@ def _write_u32(file, x):
 class Au_read:
 
     def __init__(self, f):
-        if type(f) == type(''):
+        if isinstance(f, str):
             import builtins
             f = builtins.open(f, 'rb')
             self._opened = True
@@ -312,7 +312,7 @@ class Au_read:
 class Au_write:
 
     def __init__(self, f):
-        if type(f) == type(''):
+        if isinstance(f, str):
             import builtins
             f = builtins.open(f, 'wb')
             self._opened = True
index e3f1cd81aab205259f8d5a0afa73c5e263858229..e158c19db2d65aafa8e35c3a2e31f722e9643a1a 100644 (file)
@@ -6,6 +6,9 @@ from test.pickletester import ExtensionSaver
 class C:
     pass
 
+def pickle_C(c):
+    return C, ()
+
 
 class WithoutSlots(object):
     pass
@@ -32,16 +35,15 @@ class WithInherited(WithSingleString):
 class CopyRegTestCase(unittest.TestCase):
 
     def test_class(self):
-        self.assertRaises(TypeError, copyreg.pickle,
-                          C, None, None)
+        copyreg.pickle(C, pickle_C)
 
     def test_noncallable_reduce(self):
         self.assertRaises(TypeError, copyreg.pickle,
-                          type(1), "not a callable")
+                          C, "not a callable")
 
     def test_noncallable_constructor(self):
         self.assertRaises(TypeError, copyreg.pickle,
-                          type(1), int, "not a callable")
+                          C, pickle_C, "not a callable")
 
     def test_bool(self):
         import copy
index afe0f7e9c7fd422b71ea7d770b0ea4379f608c33..9a09d8a9e3fe0360e93f4d0c9498ec9f3168e901 100644 (file)
@@ -426,7 +426,7 @@ class ClassPropertiesAndMethods(unittest.TestCase):
             def __getitem__(self, key):
                 return self.get(key, 0)
             def __setitem__(self_local, key, value):
-                self.assertIsInstance(key, type(0))
+                self.assertIsInstance(key, int)
                 dict.__setitem__(self_local, key, value)
             def setstate(self, state):
                 self.state = state
@@ -871,7 +871,7 @@ class ClassPropertiesAndMethods(unittest.TestCase):
         self.assertEqual(a.getstate(), 10)
         class D(dict, C):
             def __init__(self):
-                type({}).__init__(self)
+                dict.__init__(self)
                 C.__init__(self)
         d = D()
         self.assertEqual(list(d.keys()), [])
@@ -3288,7 +3288,7 @@ order (MRO) for bases """
         cant(True, int)
         cant(2, bool)
         o = object()
-        cant(o, type(1))
+        cant(o, int)
         cant(o, type(None))
         del o
         class G(object):
index c7b9893943471fb3cc91d71f8254d1de89cf21ff..6ea7e7db2ce1132bd99c3e40c10cc57171830706 100644 (file)
@@ -203,7 +203,7 @@ class QueryTestCase(unittest.TestCase):
     def test_unreadable(self):
         # Not recursive but not readable anyway
         pp = pprint.PrettyPrinter()
-        for unreadable in type(3), pprint, pprint.isrecursive:
+        for unreadable in object(), int, pprint, pprint.isrecursive:
             # module-level convenience functions
             self.assertFalse(pprint.isrecursive(unreadable),
                              "expected not isrecursive for %r" % (unreadable,))
index fa85c7e6cd87d5bc8880084c86e91bf26bf05182..30e843a423a777b24583a26c9364fc7d05edb4eb 100644 (file)
@@ -50,10 +50,10 @@ class CreationTestCase(unittest.TestCase):
     def testReturnType(self):
         # Test return type of gettimeout()
         self.sock.settimeout(1)
-        self.assertEqual(type(self.sock.gettimeout()), type(1.0))
+        self.assertIs(type(self.sock.gettimeout()), float)
 
         self.sock.settimeout(3.9)
-        self.assertEqual(type(self.sock.gettimeout()), type(1.0))
+        self.assertIs(type(self.sock.gettimeout()), float)
 
     def testTypeCheck(self):
         # Test type checking by settimeout()
index a614cef6ab2f1a52b3ece45f9a3c2e6920802e15..bef23f4505e03c9ff2e61337ca4ff44ce508dbfc 100644 (file)
@@ -850,9 +850,9 @@ class MultiCallIterator:
 
     def __getitem__(self, i):
         item = self.results[i]
-        if type(item) == type({}):
+        if isinstance(item, dict):
             raise Fault(item['faultCode'], item['faultString'])
-        elif type(item) == type([]):
+        elif isinstance(item, list):
             return item[0]
         else:
             raise ValueError("unexpected type in multicall result")
index 91e2dd6191e5ddb1609967950a8cec9bd9be1558..b5bffaff4ed0dcd87cd8d684d3ca40db14c8a3cf 100755 (executable)
@@ -1351,7 +1351,7 @@ def buildPython():
         build_time_vars = l_dict['build_time_vars']
     vars = {}
     for k, v in build_time_vars.items():
-        if type(v) == type(''):
+        if isinstance(v, str):
             for p in (include_path, lib_path):
                 v = v.replace(' ' + p, '')
                 v = v.replace(p + ' ', '')
index 635e5482e63c108c55a9c2c3fc072dc95d4df91b..9595ee4a01533ec038318118c83c7b06e60df7e0 100755 (executable)
@@ -70,7 +70,7 @@ emparse_list_list = [
 # compile the re's in the list and store them in-place.
 for i in range(len(emparse_list_list)):
     x = emparse_list_list[i]
-    if type(x) is type(''):
+    if isinstance(x, str):
         x = re.compile(x, re.MULTILINE)
     else:
         xl = []
@@ -105,7 +105,7 @@ def emparse_list(fp, sub):
     emails = []
     reason = None
     for regexp in emparse_list_list:
-        if type(regexp) is type(()):
+        if isinstance(regexp, tuple):
             res = regexp[0].search(data, 0, from_index)
             if res is not None:
                 try:
@@ -134,7 +134,7 @@ def emparse_list(fp, sub):
         if reason[:15] == 'returned mail: ':
             reason = reason[15:]
         for regexp in emparse_list_reason:
-            if type(regexp) is type(''):
+            if isinstance(regexp, str):
                 for i in range(len(emails)-1,-1,-1):
                     email = emails[i]
                     exp = re.compile(re.escape(email).join(regexp.split('<>')), re.MULTILINE)