diff options
author | Masahiro Yamada <yamada.masahiro@socionext.com> | 2018-11-30 04:15:48 -0500 |
---|---|---|
committer | Masahiro Yamada <yamada.masahiro@socionext.com> | 2018-12-07 20:40:52 -0500 |
commit | 75889e9be78febab9991f9c07fdf09d128562bf5 (patch) | |
tree | 8213b1bf39312fa5f4cbe7ae37a3371ef190e998 /scripts/kconfig | |
parent | a9b722847872d43595d072d7fd550f08fe6764fd (diff) |
kconfig: remove unneeded setsym label in conf_read_simple()
The two 'goto setsym' statements are reachable only when sym == NULL.
The code below the 'setsym:' label does nothing when sym == NULL
since there is just one if-block guarded by 'if (sym && ...)'.
Hence, 'goto setsym' can be replaced with 'continue'.
Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
Diffstat (limited to 'scripts/kconfig')
-rw-r--r-- | scripts/kconfig/confdata.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/scripts/kconfig/confdata.c b/scripts/kconfig/confdata.c index 91d0a5c014ac..1e35529d4eeb 100644 --- a/scripts/kconfig/confdata.c +++ b/scripts/kconfig/confdata.c | |||
@@ -363,7 +363,7 @@ load: | |||
363 | sym = sym_find(line + 2 + strlen(CONFIG_)); | 363 | sym = sym_find(line + 2 + strlen(CONFIG_)); |
364 | if (!sym) { | 364 | if (!sym) { |
365 | sym_add_change_count(1); | 365 | sym_add_change_count(1); |
366 | goto setsym; | 366 | continue; |
367 | } | 367 | } |
368 | } else { | 368 | } else { |
369 | sym = sym_lookup(line + 2 + strlen(CONFIG_), 0); | 369 | sym = sym_lookup(line + 2 + strlen(CONFIG_), 0); |
@@ -397,7 +397,7 @@ load: | |||
397 | sym = sym_find(line + strlen(CONFIG_)); | 397 | sym = sym_find(line + strlen(CONFIG_)); |
398 | if (!sym) { | 398 | if (!sym) { |
399 | sym_add_change_count(1); | 399 | sym_add_change_count(1); |
400 | goto setsym; | 400 | continue; |
401 | } | 401 | } |
402 | } else { | 402 | } else { |
403 | sym = sym_lookup(line + strlen(CONFIG_), 0); | 403 | sym = sym_lookup(line + strlen(CONFIG_), 0); |
@@ -416,7 +416,7 @@ load: | |||
416 | 416 | ||
417 | continue; | 417 | continue; |
418 | } | 418 | } |
419 | setsym: | 419 | |
420 | if (sym && sym_is_choice_value(sym)) { | 420 | if (sym && sym_is_choice_value(sym)) { |
421 | struct symbol *cs = prop_get_symbol(sym_get_choice_prop(sym)); | 421 | struct symbol *cs = prop_get_symbol(sym_get_choice_prop(sym)); |
422 | switch (sym->def[def].tri) { | 422 | switch (sym->def[def].tri) { |