From bf7aa82eab0f1f9c78d39cf57a3f6b25eefc47c1 Mon Sep 17 00:00:00 2001 From: Zack Weinberg Date: Thu, 22 Jun 2017 18:06:15 -0400 Subject: Make error_t always int; make __errno_location return an __error_t. error_t is a Hurdism whose purpose, as far as I know, is to make it so you can get GDB to tell you the E-constant corresponding to the current value of errno by typing 'p (error_t) errno'. I tried to generalize this to all platforms a few weeks ago and it was suggested to me that this is maybe a job for a GDB pretty-printer, instead. This patch clears the ground for that, by removing the Hurd-specific definition of error_t. error_t is also used to make the return values of a few GNU extension functions (in argp.h and argz.h) a little more self-documenting, so it can't completely go away, and it will be useful to be able to write 'p (error_t) err' where err is some ordinary int variable that happens to have an errno value stashed in it. So now stdlib/errno.h defines it and it's always just a typedef for 'int'. This patch also changes all definitions of __errno_location and the underlying thread-local errno to have type __error_t instead of int. __error_t is also just a typedef for int, but this is how we will cue the GDB pretty-printer to print errno specially. I believe that this does not have any standards-compliance consequences, because it's just a typedef for int. This part of the change doesn't make sense in the absence of the pretty-printer itself, but this split-up seemed more useful to reviewers. * stdlib/errno.h (__error_t): New type (typedef as int). (__errno_location): Declare as returning __error_t. (error_t): Typedef as __error_t, if not already defined. * csu/errno-loc.c, csu/errno.c, include/errno.h * sysdeps/mach/hurd/dl-sysdep.c, sysdeps/mach/hurd/errno-loc.c: Change type of errno, __libc_errno, and rtld_errno, and return type of __errno_location to __error_t. * sysdeps/mach/hudr/errnos.awk: Do not emit enum __error_t_codes or a definition of error_t. * sysdeps/mach/hurd/bits/errno.h: Regenerate. * sysdeps/mach/hurd/errno.c: Delete file. * bits/errno.h: Update commentary. --- sysdeps/mach/hurd/errno-loc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'sysdeps/mach/hurd/errno-loc.c') diff --git a/sysdeps/mach/hurd/errno-loc.c b/sysdeps/mach/hurd/errno-loc.c index 039c9fc748..0a9eae1414 100644 --- a/sysdeps/mach/hurd/errno-loc.c +++ b/sysdeps/mach/hurd/errno-loc.c @@ -19,7 +19,7 @@ #include #include -int * +__error_t * __errno_location (void) { return (int *) __hurd_threadvar_location (_HURD_THREADVAR_ERRNO); -- cgit 1.4.1