Remove outdated Python pre-3.9 code

These are manual fixes

Author: DimitriPapadopoulos

src/c/test_c.py

@@ -3,12 +3,6 @@
 from .error import CDefError
 from . import model
 
-try:
-    basestring
-except NameError:
-    # Python 3.x
-    basestring = str
-
 _unspecified = object()
 
 
@@ -111,7 +105,7 @@ def embedding_api(self, csource, packed=False, pack=None):
 
     def _cdef(self, csource, override=False, **options):
         if not isinstance(csource, str):    # unicode, on Python 2
-            if not isinstance(csource, basestring):
+            if not isinstance(csource, str):
                 raise TypeError("cdef() argument must be a string")
             csource = csource.encode('ascii')
         with self._lock:
@@ -134,7 +128,7 @@ def dlopen(self, name, flags=0):
         linked to a particular library, just like C headers; in the
         library we only look for the actual (untyped) symbols.
         """
-        if not (isinstance(name, basestring) or
+        if not (isinstance(name, str) or
                 name is None or
                 isinstance(name, self.CData)):
             raise TypeError("dlopen(name): name must be a file name, None, "
@@ -189,7 +183,7 @@ def typeof(self, cdecl):
         corresponding <ctype> object.
         It can also be used on 'cdata' instance to get its C type.
         """
-        if isinstance(cdecl, basestring):
+        if isinstance(cdecl, str):
             return self._typeof(cdecl)
         if isinstance(cdecl, self.CData):
             return self._backend.typeof(cdecl)
@@ -207,7 +201,7 @@ def sizeof(self, cdecl):
         """Return the size in bytes of the argument.  It can be a
         string naming a C type, or a 'cdata' instance.
         """
-        if isinstance(cdecl, basestring):
+        if isinstance(cdecl, str):
             BType = self._typeof(cdecl)
             return self._backend.sizeof(BType)
         else:
@@ -217,7 +211,7 @@ def alignof(self, cdecl):
         """Return the natural alignment size in bytes of the C type
         given as a string.
         """
-        if isinstance(cdecl, basestring):
+        if isinstance(cdecl, str):
             cdecl = self._typeof(cdecl)
         return self._backend.alignof(cdecl)
 
@@ -228,7 +222,7 @@ def offsetof(self, cdecl, *fields_or_indexes):
         You can also give numeric values which correspond to array
         items, in case of an array type.
         """
-        if isinstance(cdecl, basestring):
+        if isinstance(cdecl, str):
             cdecl = self._typeof(cdecl)
         return self._typeoffsetof(cdecl, *fields_or_indexes)[1]
 
@@ -255,7 +249,7 @@ def new(self, cdecl, init=None):
         about that when copying the pointer to the memory somewhere
         else, e.g. into another structure.
         """
-        if isinstance(cdecl, basestring):
+        if isinstance(cdecl, str):
             cdecl = self._typeof(cdecl)
         return self._backend.newp(cdecl, init)
 
@@ -278,7 +272,7 @@ def new_allocator(self, alloc=None, free=None,
         allocator = compiled_ffi.new_allocator(alloc, free,
                                                should_clear_after_alloc)
         def allocate(cdecl, init=None):
-            if isinstance(cdecl, basestring):
+            if isinstance(cdecl, str):
                 cdecl = self._typeof(cdecl)
             return allocator(cdecl, init)
         return allocate
@@ -288,7 +282,7 @@ def cast(self, cdecl, source):
         type initialized with the given 'source'.  The source is
         casted between integers or pointers of any type.
         """
-        if isinstance(cdecl, basestring):
+        if isinstance(cdecl, str):
             cdecl = self._typeof(cdecl)
         return self._backend.cast(cdecl, source)
 
@@ -353,7 +347,7 @@ def from_buffer(self, cdecl, python_buffer=_unspecified,
         """
         if python_buffer is _unspecified:
             cdecl, python_buffer = self.BCharA, cdecl
-        elif isinstance(cdecl, basestring):
+        elif isinstance(cdecl, str):
             cdecl = self._typeof(cdecl)
         return self._backend.from_buffer(cdecl, python_buffer,
                                          require_writable)
@@ -388,7 +382,7 @@ def callback_decorator_wrap(python_callable):
                                 "is not callable")
             return self._backend.callback(cdecl, python_callable,
                                           error, onerror)
-        if isinstance(cdecl, basestring):
+        if isinstance(cdecl, str):
             cdecl = self._typeof(cdecl, consider_function_as_funcptr=True)
         if python_callable is None:
             return callback_decorator_wrap                # decorator mode
@@ -401,7 +395,7 @@ def getctype(self, cdecl, replace_with=''):
         extra text to append (or insert for more complicated C types), like
         a variable name, or '*' to get actually the C type 'pointer-to-cdecl'.
         """
-        if isinstance(cdecl, basestring):
+        if isinstance(cdecl, str):
             cdecl = self._typeof(cdecl)
         replace_with = replace_with.strip()
         if (replace_with.startswith('*')
@@ -629,7 +623,7 @@ def set_source(self, module_name, source, source_extension='.c', **kwds):
         if hasattr(self, '_assigned_source'):
             raise ValueError("set_source() cannot be called several times "
                              "per ffi object")
-        if not isinstance(module_name, basestring):
+        if not isinstance(module_name, str):
             raise TypeError("'module_name' must be a string")
         if os.sep in module_name or (os.altsep and os.altsep in module_name):
             raise ValueError("'module_name' must not contain '/': use a dotted "
@@ -795,7 +789,7 @@ def list_types(self):
 
 def _load_backend_lib(backend, name, flags):
     import os
-    if not isinstance(name, basestring):
+    if not isinstance(name, str):
         if sys.platform != "win32" or name is not None:
             return backend.load_library(name, flags)
         name = "c"    # Windows: load_library(None) fails, but this works
@@ -931,7 +925,7 @@ def __cffi_close__(self):
             backendlib.close_lib()
             self.__dict__.clear()
     #
-    if isinstance(libname, basestring):
+    if isinstance(libname, str):
         try:
             if not isinstance(libname, str):    # unicode, on Python 2
                 libname = libname.encode('utf-8')

src/cffi/api.py

@@ -1,11 +1,6 @@
 import ctypes, ctypes.util, operator, sys
 from . import model
 
-unicode = str
-long = int
-xrange = range
-bytechr = lambda num: bytes([num])
-
 class CTypesType(type):
     pass
 
@@ -158,7 +153,7 @@ def _newp(cls, init):
         return cls(init)
 
     def __iter__(self):
-        for i in xrange(len(self)):
+        for i in range(len(self)):
             yield self[i]
 
     def _get_own_repr(self):
@@ -180,7 +175,7 @@ def _cast_from(cls, source):
             address = 0
         elif isinstance(source, CTypesData):
             address = source._cast_to_integer()
-        elif isinstance(source, (int, long)):
+        elif isinstance(source, int):
             address = source
         else:
             raise TypeError("bad type for cast to %r: %r" %
@@ -354,7 +349,7 @@ def new_primitive_type(self, name):
             is_signed = (ctype(-1).value == -1)
         #
         def _cast_source_to_int(source):
-            if isinstance(source, (int, long, float)):
+            if isinstance(source, (int, float)):
                 source = int(source)
             elif isinstance(source, CTypesData):
                 source = source._cast_to_integer()
@@ -395,7 +390,7 @@ def __int__(self):
             if kind == 'bool':
                 @classmethod
                 def _cast_from(cls, source):
-                    if not isinstance(source, (int, long, float)):
+                    if not isinstance(source, (int, float)):
                         source = _cast_source_to_int(source)
                     return cls(bool(source))
                 def __int__(self):
@@ -405,7 +400,7 @@ def __int__(self):
                 @classmethod
                 def _cast_from(cls, source):
                     source = _cast_source_to_int(source)
-                    source = bytechr(source & 0xFF)
+                    source = bytes([source & 0xFF])
                     return cls(source)
                 def __int__(self):
                     return ord(self._value)
@@ -434,7 +429,7 @@ def __float__(self):
             if kind in {'int', 'byte', 'bool'}:
                 @staticmethod
                 def _to_ctypes(x):
-                    if not isinstance(x, (int, long)):
+                    if not isinstance(x, int):
                         if isinstance(x, CTypesData):
                             x = int(x)
                         else:
@@ -467,7 +462,7 @@ def __nonzero__(self):
             if kind == 'float':
                 @staticmethod
                 def _to_ctypes(x):
-                    if not isinstance(x, (int, long, float, CTypesData)):
+                    if not isinstance(x, (int, float, CTypesData)):
                         raise TypeError("float expected, got %s" %
                                         type(x).__name__)
                     return ctype(x).value
@@ -531,14 +526,14 @@ def __init__(self, init):
                 self._own = True
 
             def __add__(self, other):
-                if isinstance(other, (int, long)):
+                if isinstance(other, int):
                     return self._new_pointer_at(self._address +
                                                 other * self._bitem_size)
                 else:
                     return NotImplemented
 
             def __sub__(self, other):
-                if isinstance(other, (int, long)):
+                if isinstance(other, int):
                     return self._new_pointer_at(self._address -
                                                 other * self._bitem_size)
                 elif type(self) is type(other):
@@ -613,7 +608,7 @@ class CTypesArray(CTypesGenericArray):
 
             def __init__(self, init):
                 if length is None:
-                    if isinstance(init, (int, long)):
+                    if isinstance(init, int):
                         len1 = init
                         init = None
                     elif kind == 'char' and isinstance(init, bytes):
@@ -692,7 +687,7 @@ def _arg_to_ctypes(value):
                 return CTypesPtr._arg_to_ctypes(value)
 
             def __add__(self, other):
-                if isinstance(other, (int, long)):
+                if isinstance(other, int):
                     return CTypesPtr._new_pointer_at(
                         ctypes.addressof(self._blob) +
                         other * ctypes.sizeof(BItem._ctype))
@@ -772,7 +767,7 @@ def initialize(blob, init):
                                     "only one supported (use a dict if needed)"
                                      % (len(init),))
             if not isinstance(init, dict):
-                if isinstance(init, (bytes, unicode)):
+                if isinstance(init, (bytes, str)):
                     raise TypeError("union initializer: got a str")
                 init = tuple(init)
                 if len(init) > len(fnames):
@@ -1057,7 +1052,7 @@ def typeoffsetof(self, BType, fieldname, num=0):
             if BField is Ellipsis:
                 raise TypeError("not supported for bitfields")
             return (BField, BType._offsetof(fieldname))
-        elif isinstance(fieldname, (int, long)):
+        elif isinstance(fieldname, int):
             if issubclass(BType, CTypesGenericArray):
                 BType = BType._CTPtr
             if not issubclass(BType, CTypesGenericPtr):

src/cffi/backend_ctypes.py

@@ -1,3 +1,4 @@
+import io
 import sys, os
 from .error import VerificationError
 
@@ -81,13 +82,6 @@ def maybe_relative_path(path):
 
 # ____________________________________________________________
 
-try:
-    int_or_long = (int, long)
-    import cStringIO
-except NameError:
-    int_or_long = int      # Python 3
-    import io as cStringIO
-
 def _flatten(x, f):
     if isinstance(x, str):
         f.write('%ds%s' % (len(x), x))
@@ -101,13 +95,13 @@ def _flatten(x, f):
         f.write('%dl' % len(x))
         for value in x:
             _flatten(value, f)
-    elif isinstance(x, int_or_long):
+    elif isinstance(x, int):
         f.write('%di' % (x,))
     else:
         raise TypeError(
             "the keywords to verify() contains unsupported object %r" % (x,))
 
 def flatten(x):
-    f = cStringIO.StringIO()
+    f = io.StringIO()
     _flatten(x, f)
     return f.getvalue()

src/cffi/ffiplatform.py

@@ -42,13 +42,12 @@ def call(libname, flag, encoding=sys.getfilesystemencoding()):
             pass
         raise PkgConfigError(berr.strip())
 
-    if sys.version_info >= (3,) and not isinstance(bout, str):   # Python 3.x
-        try:
-            bout = bout.decode(encoding)
-        except UnicodeDecodeError:
-            raise PkgConfigError("pkg-config %s %s returned bytes that cannot "
-                                 "be decoded with encoding %r:\n%r" %
-                                 (flag, libname, encoding, bout))
+	try:
+		bout = bout.decode(encoding)
+	except UnicodeDecodeError:
+		raise PkgConfigError("pkg-config %s %s returned bytes that cannot "
+							 "be decoded with encoding %r:\n%r" %
+							 (flag, libname, encoding, bout))
 
     if os.altsep != '\\' and '\\' in bout:
         raise PkgConfigError("pkg-config %s %s returned an unsupported "

src/cffi/pkgconfig.py

@@ -8,10 +8,8 @@
 VERSION_CHAR16CHAR32 = 0x2801
 
 FREE_THREADED_BUILD = sysconfig.get_config_var("Py_GIL_DISABLED")
-USE_LIMITED_API = ((sys.platform != 'win32' or sys.version_info < (3, 0) or
-                   sys.version_info >= (3, 5)) and
-                   # free-threaded build doesn't support the stable ABI until Python 3.15
-                   (not FREE_THREADED_BUILD or sys.version_info >= (3, 15)))
+# free-threaded build doesn't support the stable ABI until Python 3.15
+USE_LIMITED_API = not FREE_THREADED_BUILD or sys.version_info >= (3, 15)
 
 class GlobalExpr:
     def __init__(self, name, address, type_op, size=0, check_value=0):
@@ -1309,11 +1307,8 @@ def _print_string_literal_in_array(self, s):
             s.decode('utf-8')           # got bytes, check for valid utf-8
         for line in s.splitlines(True):
             comment = line
-            if type('//') is bytes:     # python2
-                line = map(ord, line)   #     make a list of integers
-            else:                       # python3
-                # type(line) is bytes, which enumerates like a list of integers
-                comment = ascii(comment)[1:-1]
+            # type(line) is bytes, which enumerates like a list of integers
+            comment = ascii(comment)[1:-1]
             prnt(('// ' + comment).rstrip())
             printed_line = ''
             for c in line:
@@ -1398,8 +1393,6 @@ def _emit_bytecode_EnumType(self, tp, index):
         self.cffi_types[index] = CffiOp(OP_ENUM, enum_index)
 
 
-NativeIO = io.StringIO
-
 def _is_file_like(maybefile):
     # compare to xml.etree.ElementTree._get_writer
     return hasattr(maybefile, 'write')
@@ -1414,7 +1407,7 @@ def _make_c_or_py_source(ffi, module_name, preamble, target_file, verbose):
     if _is_file_like(target_file):
         recompiler.write_source_to_f(target_file, preamble)
         return True
-    f = NativeIO()
+    f = io.StringIO()
     recompiler.write_source_to_f(f, preamble)
     output = f.getvalue()
     try:

src/cffi/recompiler.py

@@ -55,9 +55,8 @@ def write_source_to_f(self):
         # 'export_symbols', so instead of fighting it, just give up and
         # give it one
         if sys.platform == 'win32':
-            prefix = 'PyInit_'
             modname = self.verifier.get_module_name()
-            prnt("void %s%s(void) { }\n" % (prefix, modname))
+            prnt(f"void PyInit_{modname}(void) {{ }}\n")
 
     def load_library(self, flags=0):
         # import it with the CFFI backend

src/cffi/vengine_gen.py

@@ -161,7 +161,7 @@ def _write_source(self, file=None):
             self._write_source_to(file)
         else:
             # Write our source file to an in memory file.
-            f = NativeIO()
+            f = io.StringIO()
             self._write_source_to(f)
             source_data = f.getvalue()