]> git.ipfire.org Git - thirdparty/Python/cpython.git/commitdiff
Fix pyflakes warnings: variable is assigned to but never used (#142294)
authorVictor Stinner <vstinner@python.org>
Mon, 8 Dec 2025 13:00:31 +0000 (14:00 +0100)
committerGitHub <noreply@github.com>
Mon, 8 Dec 2025 13:00:31 +0000 (14:00 +0100)
Example of fixed warning:

    Lib/netrc.py:98:13: local variable 'toplevel'
    is assigned to but never used

32 files changed:
Lib/_py_warnings.py
Lib/_threading_local.py
Lib/asyncio/base_events.py
Lib/asyncio/proactor_events.py
Lib/asyncio/tools.py
Lib/asyncio/unix_events.py
Lib/codeop.py
Lib/compileall.py
Lib/concurrent/interpreters/_queues.py
Lib/dis.py
Lib/encodings/uu_codec.py
Lib/ftplib.py
Lib/functools.py
Lib/idlelib/pyshell.py
Lib/idlelib/textview.py
Lib/imaplib.py
Lib/inspect.py
Lib/modulefinder.py
Lib/multiprocessing/connection.py
Lib/netrc.py
Lib/ntpath.py
Lib/optparse.py
Lib/pickle.py
Lib/platform.py
Lib/pyclbr.py
Lib/re/_parser.py
Lib/subprocess.py
Lib/tempfile.py
Lib/tokenize.py
Lib/turtle.py
Lib/urllib/parse.py
Lib/venv/__init__.py

index 67c74fdd2d0b421da16cdc648fda2b7767baefbc..d5a9cec86f3674458eb934deec209d0b664fcf07 100644 (file)
@@ -563,7 +563,6 @@ def warn_explicit(message, category, filename, lineno,
     else:
         text = message
         message = category(message)
-    modules = None
     key = (text, category, lineno)
     with _wm._lock:
         if registry is None:
index 0b9e5d3bbf6ef6454eb5b6bff6f36eb4291d7651..2af3885458b54f2ea41c75a072f931bcea623db4 100644 (file)
@@ -57,7 +57,7 @@ class _localimpl:
             # as soon as the OS-level thread ends instead.
             local = wrlocal()
             if local is not None:
-                dct = local.dicts.pop(idt)
+                local.dicts.pop(idt)
         wrlocal = ref(self, local_deleted)
         wrthread = ref(thread, thread_deleted)
         thread.__dict__[key] = wrlocal
index 8cbb71f708537fe2c7b04586b49a6e67132a8996..6619c87bcf5b93fff2b849ae142d8087993be863 100644 (file)
@@ -949,7 +949,7 @@ class BaseEventLoop(events.AbstractEventLoop):
         try:
             return await self._sock_sendfile_native(sock, file,
                                                     offset, count)
-        except exceptions.SendfileNotAvailableError as exc:
+        except exceptions.SendfileNotAvailableError:
             if not fallback:
                 raise
         return await self._sock_sendfile_fallback(sock, file,
@@ -1270,7 +1270,7 @@ class BaseEventLoop(events.AbstractEventLoop):
             try:
                 return await self._sendfile_native(transport, file,
                                                    offset, count)
-            except exceptions.SendfileNotAvailableError as exc:
+            except exceptions.SendfileNotAvailableError:
                 if not fallback:
                     raise
 
index f404273c3ae5c1d0427ea1396fc591ed0d619b1e..3fa93b14a6787ffe8372cf56f79d89845c39e0f5 100644 (file)
@@ -733,7 +733,7 @@ class BaseProactorEventLoop(base_events.BaseEventLoop):
     async def _sock_sendfile_native(self, sock, file, offset, count):
         try:
             fileno = file.fileno()
-        except (AttributeError, io.UnsupportedOperation) as err:
+        except (AttributeError, io.UnsupportedOperation):
             raise exceptions.SendfileNotAvailableError("not a regular file")
         try:
             fsize = os.fstat(fileno).st_size
index 1d463ea09ba5b86d13779dff517ae19b2156048f..f9b8a4ee56c5c14cef5f4fda24bd4fa16700bb53 100644 (file)
@@ -244,7 +244,7 @@ def _get_awaited_by_tasks(pid: int) -> list:
             e = e.__context__
         print(f"Error retrieving tasks: {e}")
         sys.exit(1)
-    except PermissionError as e:
+    except PermissionError:
         exit_with_permission_help_text()
 
 
index 1c1458127db5ac19c8277d73a1bbdc7c4832a1dc..49e8067ee7b4e595c57db364144e65fc16652e1b 100644 (file)
@@ -359,7 +359,7 @@ class _UnixSelectorEventLoop(selector_events.BaseSelectorEventLoop):
                 "os.sendfile() is not available")
         try:
             fileno = file.fileno()
-        except (AttributeError, io.UnsupportedOperation) as err:
+        except (AttributeError, io.UnsupportedOperation):
             raise exceptions.SendfileNotAvailableError("not a regular file")
         try:
             fsize = os.fstat(fileno).st_size
index 8cac00442d99e3a47dd8505603473968904d2340..40e88423119bc4f979be06983040d4cde0d20785 100644 (file)
@@ -66,9 +66,9 @@ def _maybe_compile(compiler, source, filename, symbol, flags):
             try:
                 compiler(source + "\n", filename, symbol, flags=flags)
                 return None
-            except _IncompleteInputError as e:
+            except _IncompleteInputError:
                 return None
-            except SyntaxError as e:
+            except SyntaxError:
                 pass
                 # fallthrough
 
index 67fe370451e1efbaf97f2972d54bb316914ef842..9519a5ac16f024cd09a1e1a2724f02551adaa7d5 100644 (file)
@@ -223,7 +223,7 @@ def compile_file(fullname, ddir=None, force=False, rx=None, quiet=0,
                     cfile = importlib.util.cache_from_source(fullname)
                     opt_cfiles[opt_level] = cfile
 
-        head, tail = name[:-3], name[-3:]
+        tail = name[-3:]
         if tail == '.py':
             if not force:
                 try:
index b5cc0b8944940dc0dce5f52d2c3f08e1d369f549..ee159d7de6382722cec440c76ff73b95e4e39b14 100644 (file)
@@ -223,7 +223,7 @@ class Queue:
         while True:
             try:
                 _queues.put(self._id, obj, unboundop)
-            except QueueFull as exc:
+            except QueueFull:
                 if timeout is not None and time.time() >= end:
                     raise  # re-raise
                 time.sleep(_delay)
@@ -258,7 +258,7 @@ class Queue:
         while True:
             try:
                 obj, unboundop = _queues.get(self._id)
-            except QueueEmpty as exc:
+            except QueueEmpty:
                 if timeout is not None and time.time() >= end:
                     raise  # re-raise
                 time.sleep(_delay)
@@ -277,7 +277,7 @@ class Queue:
         """
         try:
             obj, unboundop = _queues.get(self._id)
-        except QueueEmpty as exc:
+        except QueueEmpty:
             raise  # re-raise
         if unboundop is not None:
             assert obj is None, repr(obj)
index d6d2c1386dd7858db7aa00becfc690e7d38146da..8c257d118fb23be9db22c52e8806a63c1e632661 100644 (file)
@@ -530,7 +530,6 @@ class Formatter:
         fields.append(instr.opname.ljust(_OPNAME_WIDTH))
         # Column: Opcode argument
         if instr.arg is not None:
-            arg = repr(instr.arg)
             # If opname is longer than _OPNAME_WIDTH, we allow it to overflow into
             # the space reserved for oparg. This results in fewer misaligned opargs
             # in the disassembly output.
index 4e58c62fe9ef0f37576e8c10b19bf5abc454239c..4f8704016e2131f0901b088fd52a61a7deeb5c22 100644 (file)
@@ -56,7 +56,7 @@ def uu_decode(input, errors='strict'):
             break
         try:
             data = binascii.a2b_uu(s)
-        except binascii.Error as v:
+        except binascii.Error:
             # Workaround for broken uuencoders by /Fredrik Lundh
             nbytes = (((s[0]-32) & 63) * 4 + 5) // 3
             data = binascii.a2b_uu(s[:nbytes])
index 50771e8c17c2503a49e741dad5e550d27d91be9c..640acc64f620cc9e05912d8a3fc1cd5e0be32f2b 100644 (file)
@@ -314,9 +314,9 @@ class FTP:
         port = sock.getsockname()[1] # Get proper port
         host = self.sock.getsockname()[0] # Get proper host
         if self.af == socket.AF_INET:
-            resp = self.sendport(host, port)
+            self.sendport(host, port)
         else:
-            resp = self.sendeprt(host, port)
+            self.sendeprt(host, port)
         if self.timeout is not _GLOBAL_DEFAULT_TIMEOUT:
             sock.settimeout(self.timeout)
         return sock
@@ -455,7 +455,7 @@ class FTP:
         """
         if callback is None:
             callback = print_line
-        resp = self.sendcmd('TYPE A')
+        self.sendcmd('TYPE A')
         with self.transfercmd(cmd) as conn, \
                  conn.makefile('r', encoding=self.encoding) as fp:
             while 1:
@@ -951,7 +951,7 @@ def test():
         elif file[:2] == '-d':
             cmd = 'CWD'
             if file[2:]: cmd = cmd + ' ' + file[2:]
-            resp = ftp.sendcmd(cmd)
+            ftp.sendcmd(cmd)
         elif file == '-p':
             ftp.set_pasv(not ftp.passiveserver)
         else:
index 8063eb5ffc33046a73f24a0050c43536b1af9403..836eb680ccd4d4671bb423cd82c2640a75679827 100644 (file)
@@ -687,7 +687,7 @@ def _lru_cache_wrapper(user_function, maxsize, typed, _CacheInfo):
                     # still adjusting the links.
                     root = oldroot[NEXT]
                     oldkey = root[KEY]
-                    oldresult = root[RESULT]
+                    oldresult = root[RESULT]  # noqa: F841
                     root[KEY] = root[RESULT] = None
 
                     # Now update the cache dictionary.
index 1b7c2af1a923d7059a948eed8613dfa3c53575f4..b80c8e56c9281058496d673a96cd7cd6ab0a8f31 100755 (executable)
@@ -498,7 +498,6 @@ class ModifiedInterpreter(InteractiveInterpreter):
         self.rpcclt.close()
         self.terminate_subprocess()
         console = self.tkconsole
-        was_executing = console.executing
         console.executing = False
         self.spawn_subprocess()
         try:
index 23f0f4cb5027ec317c75f1cf1ffeb626c255ec0c..0f719a06883ad726e860a70f3d39829f1dbd1b7b 100644 (file)
@@ -129,8 +129,8 @@ class ViewWindow(Toplevel):
         self.title(title)
         self.viewframe = ViewFrame(self, contents, wrap=wrap)
         self.protocol("WM_DELETE_WINDOW", self.ok)
-        self.button_ok = button_ok = Button(self, text='Close',
-                                            command=self.ok, takefocus=False)
+        self.button_ok = Button(self, text='Close',
+                                command=self.ok, takefocus=False)
         self.viewframe.pack(side='top', expand=True, fill='both')
 
         self.is_modal = modal
index c176736548188c34a97e4c10648938a238b02135..22a0afcd981519e5dbc78a27f88f20d86c700dd5 100644 (file)
@@ -808,7 +808,7 @@ class IMAP4:
         """
 
         name = 'PROXYAUTH'
-        return self._simple_command('PROXYAUTH', user)
+        return self._simple_command(name, user)
 
 
     def rename(self, oldmailbox, newmailbox):
@@ -1310,7 +1310,7 @@ class IMAP4:
 
             try:
                 self._get_response()
-            except self.abort as val:
+            except self.abort:
                 if __debug__:
                     if self.debug >= 1:
                         self.print_log()
@@ -1867,7 +1867,7 @@ if __name__ == '__main__':
 
     try:
         optlist, args = getopt.getopt(sys.argv[1:], 'd:s:')
-    except getopt.error as val:
+    except getopt.error:
         optlist, args = (), ()
 
     stream_command = None
index 8e7511b3af015f8c8bb2ef4a5b3ead52e2d35c7b..ff462750888c8879c4473dc04f2ab05a2872cc02 100644 (file)
@@ -2140,7 +2140,6 @@ def _signature_strip_non_python_syntax(signature):
 
     current_parameter = 0
     OP = token.OP
-    ERRORTOKEN = token.ERRORTOKEN
 
     # token stream always starts with ENCODING token, skip it
     t = next(token_stream)
index b115d99ab30ff15e23bce662cfdc6a75b45ea129..7fb19a5c5d1805f1981d44985b44da3c3d8a36fb 100644 (file)
@@ -400,7 +400,6 @@ class ModuleFinder:
                 yield "relative_import", (level, fromlist, name)
 
     def scan_code(self, co, m):
-        code = co.co_code
         scanner = self.scan_opcodes
         for what, args in scanner(co):
             if what == "store":
index fc00d2861260a808604b25a0b3a3408b82dcad62..64ec53884aeb5d67b733d120ca79b75dca67e46b 100644 (file)
@@ -709,8 +709,7 @@ if sys.platform == 'win32':
                 # written data and then disconnected -- see Issue 14725.
             else:
                 try:
-                    res = _winapi.WaitForMultipleObjects(
-                        [ov.event], False, INFINITE)
+                    _winapi.WaitForMultipleObjects([ov.event], False, INFINITE)
                 except:
                     ov.cancel()
                     _winapi.CloseHandle(handle)
index 2f502c1d53364f806619e112caf9ddf5800fea70..750b5071e3c65f06bbe8c0bcfe9ab1e75d02fda7 100644 (file)
@@ -95,7 +95,7 @@ class netrc:
         while 1:
             # Look for a machine, default, or macdef top-level keyword
             saved_lineno = lexer.lineno
-            toplevel = tt = lexer.get_token()
+            tt = lexer.get_token()
             if not tt:
                 break
             elif tt[0] == '#':
index 77d2bf86a5f09dc81307641fcb104255ed474bc3..7d637325240f1cbebaa91094cf4e58a88e832a5b 100644 (file)
@@ -726,7 +726,7 @@ else:
             try:
                 if _getfinalpathname(spath) == path:
                     path = spath
-            except ValueError as ex:
+            except ValueError:
                 # Unexpected, as an invalid path should not have gained a prefix
                 # at any point, but we ignore this error just in case.
                 pass
index 02ff7140882ed641e24abda2afb1aaa5c612bdcd..5ff7f74754f9c1f9f9346e14803817e777c0bc4d 100644 (file)
@@ -1372,7 +1372,7 @@ class OptionParser (OptionContainer):
         self.values = values
 
         try:
-            stop = self._process_args(largs, rargs, values)
+            self._process_args(largs, rargs, values)
         except (BadOptionError, OptionValueError) as err:
             self.error(str(err))
 
index f3025776623d2ca116c4d659cb5857a311641779..71c12c50f7f03550c29f1153b6c1d3104a90e673 100644 (file)
@@ -1169,7 +1169,6 @@ class _Pickler:
 
     def save_global(self, obj, name=None):
         write = self.write
-        memo = self.memo
 
         if name is None:
             name = getattr(obj, '__qualname__', None)
@@ -1756,7 +1755,7 @@ class _Unpickler:
         i = self.read(1)[0]
         try:
             self.append(self.memo[i])
-        except KeyError as exc:
+        except KeyError:
             msg = f'Memo value not found at index {i}'
             raise UnpicklingError(msg) from None
     dispatch[BINGET[0]] = load_binget
@@ -1765,7 +1764,7 @@ class _Unpickler:
         i, = unpack('<I', self.read(4))
         try:
             self.append(self.memo[i])
-        except KeyError as exc:
+        except KeyError:
             msg = f'Memo value not found at index {i}'
             raise UnpicklingError(msg) from None
     dispatch[LONG_BINGET[0]] = load_long_binget
index b5017dbdb022524959ec9db75fc21a701ec85e3e..3a71b669985f132bbc09ca143082bec8168b0645 100644 (file)
@@ -305,8 +305,7 @@ def _syscmd_ver(system='', release='', version='',
                                            text=True,
                                            encoding="locale",
                                            shell=True)
-        except (OSError, subprocess.CalledProcessError) as why:
-            #print('Command %s failed: %s' % (cmd, why))
+        except (OSError, subprocess.CalledProcessError):
             continue
         else:
             break
index 37f86995d6ce00013643f1b4a8aa387cebaeae72..35772cc01d0517ef2c995583735d5adfbef0447e 100644 (file)
@@ -158,7 +158,6 @@ def _readmodule(module, path, inpackage=None):
         return _readmodule(submodule, parent['__path__'], package)
 
     # Search the path for the module.
-    f = None
     if inpackage is not None:
         search_path = path
     else:
index 35ab7ede2a75a98033c08d45b0f1a181b9232cfb..bd189fe0695f801a35de2e8093cf96a1e9ea273e 100644 (file)
@@ -455,7 +455,6 @@ def _parse_sub(source, state, verbose, nested):
     items = []
     itemsappend = items.append
     sourcematch = source.match
-    start = source.tell()
     while True:
         itemsappend(_parse(source, state, verbose, nested + 1,
                            not nested and not items))
index 4d5ab6fbff0a4643edaca0d4d4eea1c670003175..17333d8c02255deb9d6e423e8e545437d511be0d 100644 (file)
@@ -1235,7 +1235,7 @@ class Popen:
             finally:
                 self._communication_started = True
             try:
-                sts = self.wait(timeout=self._remaining_time(endtime))
+                self.wait(timeout=self._remaining_time(endtime))
             except TimeoutExpired as exc:
                 exc.timeout = timeout
                 raise
index 53d14ff5c67131c4deaab7700323bc099b286981..2bc616624599927ec3673c2ed0f864f4880c777e 100644 (file)
@@ -691,7 +691,7 @@ else:
             fd, name = _mkstemp_inner(dir, prefix, suffix, flags, output_type)
             try:
                 _os.unlink(name)
-            except BaseException as e:
+            except BaseException:
                 _os.close(fd)
                 raise
             return fd
index 1f31258ce361c945f4e370a6b44efa3277911ecd..11c134482db0248a22694e5bc2ca71ce34dfa532 100644 (file)
@@ -403,7 +403,7 @@ def detect_encoding(readline):
             return None
         encoding = _get_normal_name(match.group(1).decode())
         try:
-            codec = lookup(encoding)
+            lookup(encoding)
         except LookupError:
             # This behaviour mimics the Python interpreter
             if filename is None:
index e5ce2c0a03cad6ec7f3c454ce47b8b67de2b60a1..b52d681b3af1c327e0d33ec3c1146edc748c62a9 100644 (file)
@@ -565,7 +565,7 @@ class TurtleScreenBase(object):
         """Check if the string color is a legal Tkinter color string.
         """
         try:
-            rgb = self.cv.winfo_rgb(color)
+            self.cv.winfo_rgb(color)
             ok = True
         except TK.TclError:
             ok = False
@@ -3747,7 +3747,7 @@ class RawTurtle(TPen, TNavigator):
         if action == "rot":
             angle, degPAU = data
             self._rotate(-angle*degPAU/self._degreesPerAU)
-            dummy = self.undobuffer.pop()
+            self.undobuffer.pop()
         elif action == "stamp":
             stitem = data[0]
             self.clearstamp(stitem)
index c7946d7625386db0b6f9e38019da5b102a326575..79fd6abaa1c48f43fc533d1d3ff50e3a31b9182c 100644 (file)
@@ -1062,7 +1062,7 @@ def urlencode(query, doseq=False, safe='', encoding=None, errors=None,
             else:
                 try:
                     # Is this a sufficient test for sequence-ness?
-                    x = len(v)
+                    len(v)
                 except TypeError:
                     # not a sequence
                     v = quote_via(str(v), safe, encoding, errors)
index e5addcc393a1b913cdbb3425d5cad32674ab02fa..19eddde700bcf95b70f3f419f24533d3a7160404 100644 (file)
@@ -306,7 +306,6 @@ class EnvBuilder:
             binpath = context.bin_path
             path = context.env_exe
             copier = self.symlink_or_copy
-            dirname = context.python_dir
             copier(context.executable, path)
             if not os.path.islink(path):
                 os.chmod(path, 0o755)