input/vcd: fix parse_header() return value check.

Mixing tests for both a boolean and an SR_ERR at the same time is not
really a good idea.
parse_header() actually returns a boolean so only check if it returns FALSE.

This fixes the following gcc-5 warning:

src/input/vcd.c: In function 'receive':
src/input/vcd.c:506:34: warning: logical not is only applied to the left hand side of comparison [-Wlogical-not-parentheses]
   if (!parse_header(in, in->buf) != SR_OK)
                                  ^

diff --git a/src/input/vcd.c b/src/input/vcd.c
index e14215712b75c56477342a4706de8cecee5e0cf1..f1d0b06cfad9a21add050fdea81ae086e21a24a8 100644 (file)
--- a/src/input/vcd.c
+++ b/src/input/vcd.c
@@ -503,7 +503,7 @@ static int receive(struct sr_input *in, GString *buf)
        if (!inc->got_header) {
                if (!have_header(in->buf))
                        return SR_OK;
-               if (!parse_header(in, in->buf) != SR_OK)
+               if (!parse_header(in, in->buf))
                        /* There was a header in there, but it was malformed. */
                        return SR_ERR;