This is not necessary if simpler code is used. Use the split function and drop the unnecessary []
Signed-off-by: Simon Glass <s...@chromium.org> Reviewed-by: Alper Nebi Yasak <alpernebiya...@gmail.com> Suggested-by: Alper Nebi Yasak <alpernebiya...@gmail.com> --- (no changes since v2) Changes in v2: - Add new patch to remove remove_defconfig() tools/moveconfig.py | 13 +------------ 1 file changed, 1 insertion(+), 12 deletions(-) diff --git a/tools/moveconfig.py b/tools/moveconfig.py index d4a96ef45d..ecc6e16c6c 100755 --- a/tools/moveconfig.py +++ b/tools/moveconfig.py @@ -94,17 +94,6 @@ SIZES = { RE_REMOVE_DEFCONFIG = re.compile(r'(.*)_defconfig') ### helper functions ### -def remove_defconfig(defc): - """Drop the _defconfig suffix on a string - - Args: - defc (str): String to convert - - Returns: - str: string with the '_defconfig' suffix removed - """ - return RE_REMOVE_DEFCONFIG.match(defc)[1] - def check_top_directory(): """Exit if we are not at the top of source directory.""" for fname in 'README', 'Licenses': @@ -1668,7 +1657,7 @@ def do_find_config(config_list): print(f"Error: Not in Kconfig: %s" % ' '.join(adhoc)) else: print(f'{len(out)} matches') - print(' '.join([remove_defconfig(item) for item in out])) + print(' '.join(item.split('_defconfig')[0] for item in out)) def prefix_config(cfg): -- 2.35.1.616.g0bdcbb4464-goog