diff options
author | Yusuke Endoh <mame@ruby-lang.org> | 2019-07-14 21:56:39 +0900 |
---|---|---|
committer | Yusuke Endoh <mame@ruby-lang.org> | 2019-07-14 21:58:22 +0900 |
commit | 085d0e5ccb7cecb1f761c1d3c72caeeedafc7d04 (patch) | |
tree | bf622488e19d28d18202f5efaba5be8d4afee22f | |
parent | a3493521a55e32081520be805a764fd2ad43fe7b (diff) |
ruby.c (name_match_p): remove unnecessary condition
It always returns immediately when len was decremented to zero.
So len is always positive. This change will suppress Coverity Scan
warning.
-rw-r--r-- | ruby.c | 5 |
1 files changed, 2 insertions, 3 deletions
@@ -834,7 +834,7 @@ static int name_match_p(const char *name, const char *str, size_t len) { if (len == 0) return 0; - do { + while (1) { while (TOLOWER(*str) == *name) { if (!--len || !*++str) return 1; ++name; @@ -844,8 +844,7 @@ name_match_p(const char *name, const char *str, size_t len) if (*name != '-' && *name != '_') return 0; ++name; ++str; - } while (len > 0); - return !*name; + } } #define NAME_MATCH_P(name, str, len) \ |