Use RTC_UNUSED instead of conditional compilation in BWE simulator tool.

webrtc/typedefs.h

Index: webrtc/typedefs.h
diff --git a/webrtc/typedefs.h b/webrtc/typedefs.h
index 16e10c5ea14590bc78815a6a2e4486863b8f9fa5..2ed65d66143a5546a0a0e1bda121ccc966ab0bee 100644
--- a/webrtc/typedefs.h
+++ b/webrtc/typedefs.h
@@ -80,6 +80,8 @@
 // Put after a variable that might not be used, to prevent compiler warnings:
 //   int result ATTRIBUTE_UNUSED = DoSomething();
 //   assert(result == 17);
+// Deprecated since it only works with GCC & clang. See RTC_UNUSED below.
+// TODO(terelius): Remove.
 #ifndef ATTRIBUTE_UNUSED
 #if defined(__GNUC__) || defined(__clang__)
 #define ATTRIBUTE_UNUSED __attribute__ ((__unused__))
@@ -109,4 +111,13 @@
 #endif
 #endif
 
+// Prevent compiler from warning about an unused variable.
+//   int result = DoSomething();
+//   assert(result == 17);

[nisse-webrtc, 2017/03/17 10:16:37]

I think the example is misleading, we don't use assert, and there's no need to
use RTC_UNUSED together with RTC_DCHECK.

[terelius, 2017/03/17 10:28:54]

I based this on the example for ATTRIBUTE_UNUSED. Do you have any better
suggestion?

[nisse-webrtc, 2017/03/17 10:41:42]

There's rarely a good reason to use RTC_UNUSED, so maybe it's best to just
delete the example.

If you want an example, maybe you can cook one up from how it's actually used? 

int foo = get_foo();
FOO_LOGGING(foo);  // Expanding to empty in some builds
RTC_UNUSED(foo);

[terelius, 2017/03/17 11:59:15]

I like having an example that shows how to use the macro and in particular where
it should be placed compared to ATTRIBUTE_UNUSED. I also prefer using constructs
that'll be familiar to the reader (e.g. assert) over more obscure logging macros
in the example.

I have added a note in the comment that discourages use of RTC_UNUSED though.

+//   RTC_UNUSED(result);
+#ifndef RTC_UNUSED
+#define RTC_UNUSED(x) RtcUnused(static_cast<const void*>(&x))

[nisse-webrtc, 2017/03/17 10:16:37]

Please try out this simpler definition:

  #define RTC_UNUSED(x) (static_cast<void>(x))

[terelius, 2017/03/17 10:28:54]

Already tried. It does not work on Android bots.

[nisse-webrtc, 2017/03/17 10:41:42]

Where's the failure? I see you tried 

#define RTC_UNUSED(x) static_cast<const void*>(&x)

in patchset #2, but that's quite different from what I'm suggesting.

[terelius, 2017/03/17 11:59:15]

Sorry, I misread your comment. Yeah, that seems to work.

+static inline void RtcUnused(const void* dummy) {}
+#endif  // RTC_UNUSED
+
 #endif  // WEBRTC_TYPEDEFS_H_