diff options
| author | Masahiro Yamada <masahiroy@kernel.org> | 2020-02-18 13:00:31 +0300 |
|---|---|---|
| committer | Masahiro Yamada <masahiroy@kernel.org> | 2020-03-02 15:18:19 +0300 |
| commit | 3f9070a67a94a2765e99adf0913c30b683a1b840 (patch) | |
| tree | 26899cafd8a05641a40b0d36c56a2b318f3c2c44 /scripts/basic | |
| parent | 87d660f08520ccde8569a2202ff346b376011663 (diff) | |
| download | linux-3f9070a67a94a2765e99adf0913c30b683a1b840.tar.xz | |
fixdep: remove redundant null character check
If *q is '\0', the condition (isalnum(*q) || *q == '_') is false anyway.
It is redundant to ensure non-zero *q.
Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
Diffstat (limited to 'scripts/basic')
| -rw-r--r-- | scripts/basic/fixdep.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/scripts/basic/fixdep.c b/scripts/basic/fixdep.c index ad2041817985..877ca2c88246 100644 --- a/scripts/basic/fixdep.c +++ b/scripts/basic/fixdep.c @@ -246,7 +246,7 @@ static void parse_config_file(const char *p) } p += 7; q = p; - while (*q && (isalnum(*q) || *q == '_')) + while (isalnum(*q) || *q == '_') q++; if (str_ends_with(p, q - p, "_MODULE")) r = q - 7; |