diff options
author | Peter Hutterer <peter.hutterer@who-t.net> | 2014-09-05 14:15:46 +1000 |
---|---|---|
committer | Peter Hutterer <peter.hutterer@who-t.net> | 2014-09-05 14:18:11 +1000 |
commit | 4d3d761799436e80fbcf0d99797eed35e68c90d2 (patch) | |
tree | db07f53004d6f4861be6eb4c65f9a1dc5350beed /src/synaptics.c | |
parent | 536e17c83e565ddba9c7c5a4cd613edf8378e9aa (diff) |
Shut up a coverity warning
xf86-input-synaptics-1.8.0/src/synaptics.c:498: var_compare_op: Comparing
"end_str" to null implies that "end_str" might be null.
end_str can't be null, so no need for this check and no need to get Coverity
all confused.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Keith Packard <keithp@keithp.com>
Diffstat (limited to 'src/synaptics.c')
-rw-r--r-- | src/synaptics.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/synaptics.c b/src/synaptics.c index e8a08d2..ace0ec7 100644 --- a/src/synaptics.c +++ b/src/synaptics.c @@ -495,7 +495,7 @@ set_softbutton_areas_option(InputInfoPtr pInfo, char *option_name, int offset) values[i] = value; if (next_num != end_str) { - if (end_str && *end_str == '%') { + if (*end_str == '%') { in_percent |= 1 << i; end_str++; } |