Change the ADDCARRY(x) macro to eliminate clang warning about unused
authorsfisher <sfisher@f5534014-38df-0310-8fa8-9805f1628bb7>
Wed, 19 Jan 2011 22:53:38 +0000 (22:53 +0000)
committersfisher <sfisher@f5534014-38df-0310-8fa8-9805f1628bb7>
Wed, 19 Jan 2011 22:53:38 +0000 (22:53 +0000)
result from the ADDCARRY portion of the REDUCE macro.

-#define ADDCARRY(x)  (x > 65535 ? x -= 65535 : x)
+#define ADDCARRY(x)  {if ((x) > 65535) (x) -= 65535;}

The new code is from in_cksum.c in the Tahoe/CGI port of 4.4BSD-Lite2 (we're
using the "Portable Version" copy otherwise).

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@35589 f5534014-38df-0310-8fa8-9805f1628bb7

epan/in_cksum.c

index 1170f9230a026f307966833bd1c6e2280453bc48..0b4b163cd11546e34ea6d224061c5f08249a039b 100644 (file)
@@ -51,7 +51,7 @@
  * code and should be modified for each CPU to be as fast as possible.
  */
 
-#define ADDCARRY(x)  (x > 65535 ? x -= 65535 : x)
+#define ADDCARRY(x)  {if ((x) > 65535) (x) -= 65535;}
 #define REDUCE {l_util.l = sum; sum = l_util.s[0] + l_util.s[1]; ADDCARRY(sum);}
 
 int