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.