kconfig: remove meaningless if-conditional in conf_read()
authorMasahiro Yamada <yamada.masahiro@socionext.com>
Thu, 11 Jul 2019 07:33:17 +0000 (16:33 +0900)
committerChristian Lamparter <chunkeey@gmail.com>
Fri, 27 Sep 2019 20:36:44 +0000 (22:36 +0200)
commit06b75a1e922ecfa65eae40eb17b63458439c5c11
tree5431c1050c46ad57fc63f01b90bc017b275eda56
parent1d8c043300b5152e7dd703ee3bc0926e3d206030
kconfig: remove meaningless if-conditional in conf_read()

sym_is_choice(sym) has already been checked by previous if-block:

    if (sym_is_choice(sym) || (sym->flags & SYMBOL_NO_WRITE))
            continue;

Hence, the following code is redundant, and the comment is misleading:

    if (!sym_is_choice(sym))
            continue;
    /* fall through */

It always takes 'continue', never falls though.

Clean up the dead code.

Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
Signed-off-by: Christian Lamparter <chunkeey@gmail.com>
config/confdata.c