]> xenbits.xensource.com Git - libvirt.git/commitdiff
python: fix use of undeclared variables in python scripts
authorDaniel P. Berrangé <berrange@redhat.com>
Tue, 24 Sep 2019 12:10:46 +0000 (13:10 +0100)
committerDaniel P. Berrangé <berrange@redhat.com>
Mon, 11 Nov 2019 14:24:19 +0000 (14:24 +0000)
docs/apibuild.py:2436:65: F821 undefined name 'first_letter'
                        chunks.append(["chunk%s" % (chunk - 1), first_letter, letter])
                                                                ^
src/hyperv/hyperv_wmi_generator.py:415:57: F821 undefined name 'number'
        report_error("line %d: invalid block header" % (number))
                                                        ^

Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
build-aux/syntax-check.mk
docs/apibuild.py
src/hyperv/hyperv_wmi_generator.py

index 492cc79b7285ea58c1b369488584a447505b28b7..221c581a31bc005f20f3e8ed55a512849e2209c0 100644 (file)
@@ -888,7 +888,6 @@ FLAKE8_WHITESPACE = E211,E221,E222,E225,E226,E231,E261
 FLAKE8_BLANK_LINES = E301,E302,E303,E305
 FLAKE8_LINE_LENGTH = E501
 FLAKE8_STATEMENTS = E722,E741
-FLAKE8_ERRORS = F821
 FLAKE8_WARNINGS = W504,W605
 
 FLAKE8_IGNORE = $(FLAKE8_INDENTATION),$\
@@ -896,7 +895,6 @@ FLAKE8_IGNORE = $(FLAKE8_INDENTATION),$\
                $(FLAKE8_BLANK_LINES),$\
                $(FLAKE8_LINE_LENGTH),$\
                $(FLAKE8_STATEMENTS),$\
-               $(FLAKE8_ERRORS),$\
                $(FLAKE8_WARNINGS) \
                $(NULL)
 
index 26bdb7e89081c22c315ce114b8bd1ce834b911e1..20ce926ae88b894ce1a889f2894b83c18deecb5c 100755 (executable)
@@ -2401,6 +2401,7 @@ class docBuilder:
         count = 0
         chunk = 0
         chunks = []
+        first_letter = None
         for id in typ:
             if len(index[id]) > 30:
                 continue
index c3cd39cc766089cff0fcff58ff7bd90bf0f41b87..fe0ba66759e77d8ea57bd094ede43c73ab2c5079 100755 (executable)
@@ -407,7 +407,7 @@ def report_error(message):
 
 
 
-def parse_class(block):
+def parse_class(block, number):
     # expected format: class <name>
     header_items = block[0][1].split()
 
@@ -488,7 +488,7 @@ def main():
         if block is not None:
             if line == "end":
                 if block[0][1].startswith("class"):
-                    parse_class(block)
+                    parse_class(block, number)
 
                 block = None
             else: