python: Do not explicitly state variables are global when only read

If a variable is not modified in a scope there is no need for the use of
global in such scope.  Without this patch build fails with:

F824 `global ...` is unused: name is never assigned in scope

It is a bit difficult to find more information on that message and error
code, I found it here:

    https://docs.astral.sh/ruff/rules/global-variable-not-assigned/

Signed-off-by: Martin Kletzander <mkletzan@redhat.com>
Reviewed-by: Jiri Denemark <jdenemar@redhat.com>

diff --git a/scripts/apibuild.py b/scripts/apibuild.py
index 3ecc3eadf7b2d02ce9ad6ac43f00b6125cd86d56..a30e87969aa6f18d5633efb53868953d0a16a857 100755 (executable)
--- a/scripts/apibuild.py
+++ b/scripts/apibuild.py
@@ -818,8 +818,6 @@ class CParser:
     # Parse a comment block associate to a macro
     #
     def parseMacroComment(self, name, quiet=0):
-        global ignored_macros
-
         if name[0:2] == '__':
             quiet = 1
         if name in ignored_macros:
@@ -895,8 +893,6 @@ class CParser:
     # as possible
     #
     def mergeFunctionComment(self, name, description, quiet=0):
-        global ignored_functions
-
         if name == 'main':
             quiet = 1
         if name[0:2] == '__':
@@ -1135,8 +1131,6 @@ class CParser:
         self.lexer.push(tok)
 
     def token(self):
-        global ignored_words
-
         token = self.lexer.token()
         while token is not None:
             if token[0] == 'comment':

diff --git a/tools/virt-qemu-qmp-proxy b/tools/virt-qemu-qmp-proxy
index dcdb24b9a9eb2a7eacfb416f0e2a1ba56d550289..39e98e9d514727ec1dfe5aa06350b89512c9ac04 100755 (executable)
--- a/tools/virt-qemu-qmp-proxy
+++ b/tools/virt-qemu-qmp-proxy
@@ -278,7 +278,6 @@ class QMPProxy(object):
             else:
                 error = True
         except Exception as e:
-            global debug
             if debug:
                 print("%s: %s" % (sys.argv[0], str(e)))
                 print(traceback.format_exc())