GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

kconfig_lint: put $inside_choice together right in the first place 

Instead of substituting the delimiter later, put $inside_choice together
right in the first place.

Signed-off-by: Michael Niewöhner <foss@mniewoehner.de>
Change-Id: Ia713510a683101c48c86a1c3722ebb1607a29288
Reviewed-on: https://review.coreboot.org/c/coreboot/+/58400
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Martin Roth <martinroth@google.com>
This commit is contained in:
Michael Niewöhner 2021-10-17 17:43:53 +02:00 committed by Felix Held
parent a76e647094
commit 70fb5cb514
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
util/lint/kconfig_lint
View File

@ -614,7 +614,7 @@ sub build_and_parse_kconfig_tree {
$choice_symbol = $symbol; $choice_symbol = $symbol;
} }
$inside_config = ""; $inside_config = "";
$inside_choice = "$filename $line_no"; $inside_choice = "$filename:$line_no";
$configs_inside_choice = 0; $configs_inside_choice = 0;
# Kconfig verifies that choice blocks have a prompt # Kconfig verifies that choice blocks have a prompt
@ -806,7 +806,7 @@ sub add_symbol {
$symbols{$symbol}{choice} = 1; $symbols{$symbol}{choice} = 1;
# remember the location of the choice # remember the location of the choice
$symbols{$symbol}{choice_loc} = join(':', (split / /, $inside_choice)); $symbols{$symbol}{choice_loc} = $inside_choice;
} }
else { else {
$symbols{$symbol}{choice} = 0; $symbols{$symbol}{choice} = 0;