From d487c8c7e72666d5e926c4c6a499a0821796c694 Mon Sep 17 00:00:00 2001
From: Gerald Combs <gerald@wireshark.org>
Date: Mon, 30 Nov 2015 14:24:19 -0800
Subject: SpeexDSP: Work around self assignments.

Disable -Wself-assign in speex/resample.c. Many macros (particularly
SATURATE32PSHR) simply return the first argument.

Change-Id: I3a8557833343e7b213031359e94af90d32d9f082
Reviewed-on: https://code.wireshark.org/review/12315
Petri-Dish: Gerald Combs <gerald@wireshark.org>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Gerald Combs <gerald@wireshark.org>
---
 codecs/speex/resample.c | 6 ++++++
 1 file changed, 6 insertions(+)

(limited to 'codecs')

diff --git a/codecs/speex/resample.c b/codecs/speex/resample.c
index 0a903792f0..148565d028 100644
--- a/codecs/speex/resample.c
+++ b/codecs/speex/resample.c
@@ -59,6 +59,8 @@
 
 #include "config.h"
 
+#include "wsutil/ws_diag_control.h"
+
 #define OUTSIDE_SPEEX 1
 #define FLOATING_POINT 1
 
@@ -267,6 +269,8 @@ int main(int argc, char **argv)
 }
 #endif
 
+DIAG_OFF(self-assign) /* SATURATE32PSHR */
+
 #ifdef FIXED_POINT
 /* The slow way of computing a sinc for the table. Should improve that some day */
 static spx_word16_t sinc(float cutoff, float x, int N, const struct FuncDef *window_func)
@@ -1201,3 +1205,5 @@ EXPORT const char *speex_resampler_strerror(int err)
          return "Unknown error. Bad error code or strange version mismatch.";
    }
 }
+
+DIAG_ON(self-assign)
-- 
cgit v1.2.3