From 29519c447665068e3f40cdb9699e6ec39561a17e Mon Sep 17 00:00:00 2001 From: Gerald Combs Date: Wed, 2 Apr 2014 12:51:52 -0700 Subject: Remove extra parenthesis in EQ_SEQ. Fixes the following clang warning: epan/follow.c:397:20: error: equality comparison with extraneous parentheses [-Werror,-Wparentheses-equality] if ( ((sequence) == (seq[src_index])) ) { ~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~ epan/follow.c:397:20: note: remove extraneous parentheses around the comparison to silence this warning if ( ((sequence) == (seq[src_index])) ) { ~ ^ ~ epan/follow.c:397:20: note: use '=' to turn this equality comparison into an assignment if ( ((sequence) == (seq[src_index])) ) { ^~ = Change-Id: Ic257bbc598e17f854b671056887ca1a13dcea850 Reviewed-on: https://code.wireshark.org/review/928 Reviewed-by: Gerald Combs --- epan/dissectors/packet-tcp.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'epan/dissectors/packet-tcp.h') diff --git a/epan/dissectors/packet-tcp.h b/epan/dissectors/packet-tcp.h index 0c79700bc5..c7a46f3788 100644 --- a/epan/dissectors/packet-tcp.h +++ b/epan/dissectors/packet-tcp.h @@ -52,7 +52,7 @@ extern "C" { #define LT_SEQ(x, y) ((gint32)((x) - (y)) < 0) #define GE_SEQ(x, y) ((gint32)((y) - (x)) <= 0) #define LE_SEQ(x, y) ((gint32)((x) - (y)) <= 0) -#define EQ_SEQ(x, y) ((x) == (y)) +#define EQ_SEQ(x, y) (x) == (y) /* the tcp header structure, passed to tap listeners */ typedef struct tcpheader { -- cgit v1.2.3