[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
relocatable-prog: fix gcc warning on mingw
From: |
Bruno Haible |
Subject: |
relocatable-prog: fix gcc warning on mingw |
Date: |
Sat, 27 Apr 2019 01:27:45 +0200 |
User-agent: |
KMail/5.1.3 (Linux/4.4.0-141-generic; KDE/5.18.0; x86_64; ; ) |
On mingw, I'm seeing this warning:
progreloc.c:156:1: warning: ‘maybe_executable’ defined but not used
[-Wunused-function]
maybe_executable (const char *filename)
^~~~~~~~~~~~~~~~
This patch should fix it.
2019-04-26 Bruno Haible <address@hidden>
relocatable-prog: Fix gcc warning on mingw.
* lib/progreloc.c (maybe_executable): Don't define on native Windows and
on EMX.
diff --git a/lib/progreloc.c b/lib/progreloc.c
index b3eb503..9d81efa 100644
--- a/lib/progreloc.c
+++ b/lib/progreloc.c
@@ -151,17 +151,20 @@ full_read (int fd, void *buf, size_t count)
static int executable_fd = -1;
#endif
+/* Define this function only when it's needed. */
+#if !(defined WINDOWS_NATIVE || defined __EMX__)
+
/* Tests whether a given pathname may belong to the executable. */
static bool
maybe_executable (const char *filename)
{
/* The native Windows API lacks the access() function. */
-#if !defined WINDOWS_NATIVE
+# if !defined WINDOWS_NATIVE
if (access (filename, X_OK) < 0)
return false;
-#endif
+# endif
-#if defined __linux__ || defined __CYGWIN__
+# if defined __linux__ || defined __CYGWIN__
if (executable_fd >= 0)
{
/* If we already have an executable_fd, check that filename points to
@@ -179,11 +182,13 @@ maybe_executable (const char *filename)
return false;
}
}
-#endif
+# endif
return true;
}
+#endif
+
/* Determine the full pathname of the current executable, freshly allocated.
Return NULL if unknown.
Guaranteed to work on Linux and native Windows. Likely to work on the
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- relocatable-prog: fix gcc warning on mingw,
Bruno Haible <=