summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPeter Hutterer <peter.hutterer@who-t.net>2014-09-05 14:15:46 +1000
committerPeter Hutterer <peter.hutterer@who-t.net>2014-09-05 14:18:11 +1000
commit4d3d761799436e80fbcf0d99797eed35e68c90d2 (patch)
treedb07f53004d6f4861be6eb4c65f9a1dc5350beed
parent536e17c83e565ddba9c7c5a4cd613edf8378e9aa (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>
-rw-r--r--src/synaptics.c2
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++;
}