]> Savannah Git Hosting - gnulib.git/commitdiff
gnulib-tool.py: Don't initialize local variables too early.
authorBruno Haible <bruno@clisp.org>
Sat, 6 Aug 2022 01:18:32 +0000 (03:18 +0200)
committerBruno Haible <bruno@clisp.org>
Sat, 6 Aug 2022 01:18:32 +0000 (03:18 +0200)
* gnulib-tool.py (main): Initialize each local variable in one place.
I prefer to have pylint tell me about uninitialized variables than to
get the value None at runtime.

ChangeLog
gnulib-tool.py

index fde81debeab2ae0caa788fa60f73e1a811302e21..dc968b8982c08777e2671507519076bd816bb1f1 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,10 @@
 2022-08-05  Bruno Haible  <bruno@clisp.org>
 
+       gnulib-tool.py: Don't initialize local variables too early.
+       * gnulib-tool.py (main): Initialize each local variable in one place.
+       I prefer to have pylint tell me about uninitialized variables than to
+       get the value None at runtime.
+
        gnulib-tool.py: Finish implementing option --po-domain.
        * gnulib-tool.py (main): Accept option --po-domain.
        * pygnulib/GLEmiter.py (GLEmiter.po_POTFILES_in): Fix variable
index 8dbf2d06a8a4ce71c3adaf1d0efefdacc0feed44..78bb326a695fc42dc62377a0dff953dc56778a5d 100755 (executable)
@@ -67,32 +67,6 @@ isfile = os.path.isfile
 # Define main part
 #===============================================================================
 def main():
-    # Reset arguments
-    mode = None
-    destdir = None
-    localpath = None
-    verbose = None
-    auxdir = None
-    modules = None
-    avoids = None
-    sourcebase = None
-    m4base = None
-    pobase = None
-    docbase = None
-    testsbase = None
-    tests = None
-    libname = None
-    lgpl = None
-    makefile_name = None
-    libtool = None
-    conddeps = None
-    macro_prefix = None
-    podomain = None
-    witness_c_macro = None
-    vc_files = None
-    dryrun = None
-    errors = None
-
     info = classes.GLInfo()
     parser = argparse.ArgumentParser(
         prog=constants.APP['name'],
@@ -467,6 +441,9 @@ def main():
         sys.exit(1)
 
     # Determine selected mode.
+    mode = None
+    modules = None
+    files = None
     if cmdargs.mode_list != None:
         mode = 'list'
     if cmdargs.mode_import != None:
@@ -671,6 +648,10 @@ def main():
     symlink = cmdargs.symlink == True
     lsymlink = cmdargs.lsymlink == True
     single_configure = cmdargs.single_configure
+    docbase = None
+    conddeps = None
+    witness_c_macro = None
+    vc_files = None
 
     # Create pygnulib configuration.
     config = classes.GLConfig(