Consolidate duplicated patterns

This extracts shared helpers to eliminate copy-pasted logic:
- kconfiglib.py: _run_cmd()/_run_cmd_in_tmpdir() replace 9 duplicated
  subprocess functions; merge select/imply sanity loops; flatten nested
  ternaries in expr_value() and _parse_block()
- menuconfig.py: _draw_scrollbar(), _create_shadow_for_win(),
  _info_dialog_title() deduplicate rendering code; rewrite _draw_box()
  from O(area) to O(perimeter); remove dead _safe_hline/_safe_vline
- guiconfig.py: _save_dialog(), _choice_sym_prompt() helpers; data-driven
  _select_imply_info() loop; delegate ThemeManager.toggle() to set_theme
- lint.py: print_results() helper; list comprehensions replace loops
- testsuite.py: run_conf_and_compare() helper for 5 test functions
- genconfig.py: flatten nested if/else for header_path

Author: jserv

alldefconfig.py

@@ -14,6 +14,7 @@
 
   $ make [ARCH=<arch>] scriptconfig SCRIPT=Kconfiglib/alldefconfig.py
 """
+
 import kconfiglib
 
 

allmodconfig.py

@@ -13,6 +13,7 @@
 
   $ make [ARCH=<arch>] scriptconfig SCRIPT=Kconfiglib/allmodconfig.py
 """
+
 import kconfiglib
 
 

allyesconfig.py

@@ -13,6 +13,7 @@
 
   $ make [ARCH=<arch>] scriptconfig SCRIPT=Kconfiglib/allyesconfig.py
 """
+
 import kconfiglib
 
 

defconfig.py

@@ -12,6 +12,7 @@
 The default output filename is '.config'. A different filename can be passed in
 the KCONFIG_CONFIG environment variable.
 """
+
 import argparse
 
 import kconfiglib

genconfig.py

@@ -32,13 +32,13 @@
 handy way to define multi-line variables in makefiles, for use with custom
 headers. Remember to export the variable to the environment.
 """
+
 import argparse
 import os
 import sys
 
 import kconfiglib
 
-
 DEFAULT_SYNC_DEPS_PATH = "deps/"
 
 
@@ -79,9 +79,7 @@ def main():
 Enable generation of symbol dependency information for incremental builds,
 optionally specifying the output directory (default: {}). See the docstring of
 Kconfig.sync_deps() in Kconfiglib for more information.
-""".format(
-            DEFAULT_SYNC_DEPS_PATH
-        ),
+""".format(DEFAULT_SYNC_DEPS_PATH),
     )
 
     parser.add_argument(
@@ -119,17 +117,16 @@ def main():
     kconf = kconfiglib.Kconfig(args.kconfig, suppress_traceback=True)
     kconf.load_config()
 
-    if args.header_path is None:
-        if "KCONFIG_AUTOHEADER" in os.environ:
-            kconf.write_autoconf()
-        else:
-            # Kconfiglib defaults to include/generated/autoconf.h to be
-            # compatible with the C tools. 'config.h' is used here instead for
-            # backwards compatibility. It's probably a saner default for tools
-            # as well.
-            kconf.write_autoconf("config.h")
-    else:
+    if args.header_path is not None:
         kconf.write_autoconf(args.header_path)
+    elif "KCONFIG_AUTOHEADER" in os.environ:
+        kconf.write_autoconf()
+    else:
+        # Kconfiglib defaults to include/generated/autoconf.h to be
+        # compatible with the C tools. 'config.h' is used here instead for
+        # backwards compatibility. It's probably a saner default for tools
+        # as well.
+        kconf.write_autoconf("config.h")
 
     if args.config_out is not None:
         kconf.write_config(args.config_out, save_old=False)