Wireshark-dev: Re: [Wireshark-dev] #ifdef mess

From: João Valverde <joao.valverde@xxxxxxxxxxxxxxxxxx>
Date: Tue, 29 Mar 2016 15:34:38 +0100


On 28-03-2016 23:30, Joerg Mayer wrote:
Hello list,

I've been meaning to write this mail for some years now but finally got around to it.

Earlier today I committed 30900b443b85a7e760d703ca3d6efe61df4fe623, which I'm
incredibly unproud of because of readablity:

  static void
-get_reordercap_runtime_info(GString *str _U_)
+get_reordercap_runtime_info(
+#if defined(HAVE_LIBZ) && !defined(_WIN32)
+    GString *str)
+#else
+   GString *str _U_)
+#endif
  {

It fixes the error at hand, but that is about all the good I can say about it.

It's only an error because -Werror=used-but-marked-unused was enabled. Since the semantics of _U_ are *possibly* unused variable, neither the warning nor the #ifdef should exist IMO.