fixdep: remove redundant null character check
authorMasahiro Yamada <masahiroy@kernel.org>
Tue, 18 Feb 2020 10:00:31 +0000 (19:00 +0900)
committerMasahiro Yamada <masahiroy@kernel.org>
Mon, 2 Mar 2020 12:18:19 +0000 (21:18 +0900)
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>
scripts/basic/fixdep.c

index ad2041817985c6205e78b9b41794ae44e697dba3..877ca2c88246303fca7d792cc6a17ef72bfcc254 100644 (file)
@@ -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;