diff options
author | Masahiro Yamada <masahiroy@kernel.org> | 2022-02-14 12:19:18 +0900 |
---|---|---|
committer | Masahiro Yamada <masahiroy@kernel.org> | 2022-02-18 11:36:17 +0900 |
commit | a7d4f58e99dd3f6067606115ce147c15c17b6e93 (patch) | |
tree | dc0b210afb7fd50abc984506a43e419eb91e41cd | |
parent | 868653f421cd37e8ec3880da19f0aac93f5c46cc (diff) | |
download | linux-a7d4f58e99dd3f6067606115ce147c15c17b6e93.tar.gz linux-a7d4f58e99dd3f6067606115ce147c15c17b6e93.tar.bz2 linux-a7d4f58e99dd3f6067606115ce147c15c17b6e93.zip |
kconfig: fix missing '# end of' for empty menu
Currently, "# end of ..." is inserted when the menu goes back to its
parent.
Hence, an empty menu:
menu "Foo"
endmenu
... ends up with unbalanced menu comments, like this:
#
# Foo
#
Let's close the menu comments properly:
#
# Foo
#
# end of Foo
Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
-rw-r--r-- | scripts/kconfig/confdata.c | 25 |
1 files changed, 13 insertions, 12 deletions
diff --git a/scripts/kconfig/confdata.c b/scripts/kconfig/confdata.c index 94dcec2cc803..901835a56e89 100644 --- a/scripts/kconfig/confdata.c +++ b/scripts/kconfig/confdata.c @@ -903,19 +903,20 @@ next: menu = menu->list; continue; } - if (menu->next) + +end_check: + if (!menu->sym && menu_is_visible(menu) && menu != &rootmenu && + menu->prompt->type == P_MENU) { + fprintf(out, "# end of %s\n", menu_get_prompt(menu)); + need_newline = true; + } + + if (menu->next) { menu = menu->next; - else while ((menu = menu->parent)) { - if (!menu->sym && menu_is_visible(menu) && - menu != &rootmenu) { - str = menu_get_prompt(menu); - fprintf(out, "# end of %s\n", str); - need_newline = true; - } - if (menu->next) { - menu = menu->next; - break; - } + } else { + menu = menu->parent; + if (menu) + goto end_check; } } fclose(out); |