path: root/src/legacy
diff options
authorRich Felker <>2018-09-12 00:08:09 -0400
committerRich Felker <>2018-09-12 14:34:37 -0400
commit5ce3737931bb411a8d167356d4d0287b53b0cbdc (patch)
tree726fc5dde9cc462316faa715158c38f0da72292d /src/legacy
parent0676c3a34c7bf12b33f8f5efb92476f4ffc7f20e (diff)
reduce spurious inclusion of libc.h
libc.h was intended to be a header for access to global libc state and related interfaces, but ended up included all over the place because it was the way to get the weak_alias macro. most of the inclusions removed here are places where weak_alias was needed. a few were recently introduced for hidden. some go all the way back to when libc.h defined CANCELPT_BEGIN and _END, and all (wrongly implemented) cancellation points had to include it. remaining spurious users are mostly callers of the LOCK/UNLOCK macros and files that use the LFS64 macro to define the awful *64 aliases. in a few places, new inclusion of libc.h is added because several internal headers no longer implicitly include libc.h. declarations for __lockfile and __unlockfile are moved from libc.h to stdio_impl.h so that the latter does not need libc.h. putting them in libc.h made no sense at all, since the macros in stdio_impl.h are needed to use them correctly anyway.
Diffstat (limited to 'src/legacy')
2 files changed, 0 insertions, 2 deletions
diff --git a/src/legacy/euidaccess.c b/src/legacy/euidaccess.c
index 73072513..6e1f3985 100644
--- a/src/legacy/euidaccess.c
+++ b/src/legacy/euidaccess.c
@@ -1,7 +1,6 @@
#define _GNU_SOURCE
#include <unistd.h>
#include <fcntl.h>
-#include "libc.h"
int euidaccess(const char *filename, int amode)
diff --git a/src/legacy/utmpx.c b/src/legacy/utmpx.c
index 3c983679..7aa65da3 100644
--- a/src/legacy/utmpx.c
+++ b/src/legacy/utmpx.c
@@ -2,7 +2,6 @@
#include <utmpx.h>
#include <stddef.h>
#include <errno.h>
-#include "libc.h"
void endutxent(void)