projects / platform / kernel / linux-rpi.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: baa23ec) | patch
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)
committerMasahiro Yamada <yamada.masahiro@socionext.com>
Thu, 11 Jul 2019 14:37:55 +0000 (23:37 +0900)
commite3cd5136a4ecece6a7599a07add0dfb811a7f49d
tree3893506f241a8fe9c2d3b284f70386d4046ef912tree | snapshot
parentbaa23ec860920ebf3e897c4bbb3420a88ea80ec1commit | diff
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>
scripts/kconfig/confdata.c diff | blob | history
Domain: System / Kernel;