patch 8.2.4347: in some build setups UNUSED is not defined

Problem:    In some build setups UNUSED is not defined.
Solution:   Change the logic of how UNUSED is defined. (Ola Söder,
            closes #9734)

diff --git a/src/version.c b/src/version.c
index 07e46ae95ec1fad8204e504d49cc32130094123a..f7599c0d082ddcfb2118fec1e445fed81784bcf9 100644 (file)
--- a/src/version.c
+++ b/src/version.c
@@ -746,6 +746,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    4347,
 /**/
     4346,
 /**/

diff --git a/src/vim.h b/src/vim.h
index 166edf579f34b873453fe205f3ea7597308ad1dd..4f95bb45f27daf26ede2d39643048612f2ec3d01 100644 (file)
--- a/src/vim.h
+++ b/src/vim.h
@@ -245,12 +245,19 @@
 
 // Mark unused function arguments with UNUSED, so that gcc -Wunused-parameter
 // can be used to check for mistakes.
-#if defined(HAVE_ATTRIBUTE_UNUSED) || defined(__MINGW32__)
-# if !defined(UNUSED)
+#ifndef UNUSED
+# if defined(HAVE_ATTRIBUTE_UNUSED) || defined(__MINGW32__)
 #  define UNUSED __attribute__((unused))
+# else
+#  if defined __has_attribute
+#   if __has_attribute(unused)
+#    define UNUSED __attribute__((unused))
+#   endif
+#  endif
+# endif
+# ifndef UNUSED
+#  define UNUSED
 # endif
-#else
-# define UNUSED
 #endif
 
 // Used to check for "sun", "__sun" is used by newer compilers.