diff options
author | wm4 <wm4@nowhere> | 2014-11-21 05:12:55 +0100 |
---|---|---|
committer | wm4 <wm4@nowhere> | 2014-11-21 05:18:09 +0100 |
commit | 85fb2af369e9ea8638beaf3a34865e3553329a02 (patch) | |
tree | 5d73b2ea57a66fc8e9ef33358a4aa89071a2e60f /player/configfiles.c | |
parent | 4704fab82c084dbcca52b5e75f7a36877921dbd9 (diff) | |
download | mpv-85fb2af369e9ea8638beaf3a34865e3553329a02.tar.bz2 mpv-85fb2af369e9ea8638beaf3a34865e3553329a02.tar.xz |
Remove some superfluous NULL checks
In all of these situations, NULL is logically not allowed, making the
checks redundant.
Coverity complained about accessing the pointers before checking them
for NULL later.
Diffstat (limited to 'player/configfiles.c')
-rw-r--r-- | player/configfiles.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/player/configfiles.c b/player/configfiles.c index 75011eb9fa..2d966eb7f7 100644 --- a/player/configfiles.c +++ b/player/configfiles.c @@ -268,7 +268,7 @@ static bool needs_config_quoting(const char *s) { if (s[0] == '%') return true; - for (int i = 0; s && s[i]; i++) { + for (int i = 0; s[i]; i++) { unsigned char c = s[i]; if (!mp_isprint(c) || mp_isspace(c) || c == '#' || c == '\'' || c == '"') return true; |