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 | |
| 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')
| -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) { |
