[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH] faccessat: remove unused wrappers
From: |
Eric Blake |
Subject: |
[PATCH] faccessat: remove unused wrappers |
Date: |
Tue, 5 Oct 2010 08:08:31 -0600 |
* lib/openat.h (accessat, euidaccesat): Delete, since the mere
presence of these wrappers dragged in -lgen on Solaris.
Reported by Clemens Brogi; fix suggested by Paul Eggert.
Signed-off-by: Eric Blake <address@hidden>
---
I'm pushing this.
ChangeLog | 7 +++++++
lib/openat.h | 18 +++---------------
2 files changed, 10 insertions(+), 15 deletions(-)
diff --git a/ChangeLog b/ChangeLog
index edc7c8b..87559f4 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2010-10-05 Eric Blake <address@hidden>
+
+ faccessat: remove unused wrappers
+ * lib/openat.h (accessat, euidaccesat): Delete, since the mere
+ presence of these wrappers dragged in -lgen on Solaris.
+ Reported by Clemens Brogi; fix suggested by Paul Eggert.
+
2010-10-05 Jim Meyering <address@hidden>
tests: require @PRAGMA_COLUMNS@ with each @PRAGMA_SYSTEM_HEADER@
diff --git a/lib/openat.h b/lib/openat.h
index 11a0314..1d4432e 100644
--- a/lib/openat.h
+++ b/lib/openat.h
@@ -93,22 +93,10 @@ lstatat (int fd, char const *name, struct stat *st)
return fstatat (fd, name, st, AT_SYMLINK_NOFOLLOW);
}
-#if GNULIB_FACCESSAT
/* For now, there are no wrappers named laccessat or leuidaccessat,
since gnulib doesn't support faccessat(,AT_SYMLINK_NOFOLLOW) and
- since access rights on symlinks are of limited utility. */
-
-static inline int
-accessat (int fd, char const *file, int mode)
-{
- return faccessat (fd, file, mode, 0);
-}
-
-static inline int
-euidaccessat (int fd, char const *file, int mode)
-{
- return faccessat (fd, file, mode, AT_EACCESS);
-}
-#endif
+ since access rights on symlinks are of limited utility. Likewise,
+ wrappers are not provided for accessat or euidaccessat, so as to
+ avoid dragging in -lgen on some platforms. */
#endif /* _GL_HEADER_OPENAT */
--
1.7.2.3