diff options
-rw-r--r-- | lib/ChangeLog | 9 | ||||
-rw-r--r-- | lib/getopt.c | 11 | ||||
-rw-r--r-- | lib/getopt1.c | 11 | ||||
-rw-r--r-- | lib/getopt_.h | 15 |
4 files changed, 39 insertions, 7 deletions
diff --git a/lib/ChangeLog b/lib/ChangeLog index 09128a69b..f447f86ca 100644 --- a/lib/ChangeLog +++ b/lib/ChangeLog @@ -1,5 +1,14 @@ 2004-11-16 Paul Eggert <eggert@cs.ucla.edu> + Further getopt changes to make it more likely that glibc will + buy the changes back. + * getopt.c (POSIXLY_CORRECT): New constant. + (getopt): Use it, so to preserve glibc semantic + * getopt1.c (getopt_long, getopt_long_only): Arg is char * const * + when compiling for libc. + * getopt_.h (__getopt_argv_const): Bring it back. + (getopt_long, getopt_long_only): Use it. + * getopt.c (_getopt_initialize, _getopt_internal_r, _getopt_internal): New arg POSIXLY_CORRECT. All callers changed. (getopt): Argv is now char * const *, as per standard. diff --git a/lib/getopt.c b/lib/getopt.c index 744f11563..ef790b79e 100644 --- a/lib/getopt.c +++ b/lib/getopt.c @@ -1154,10 +1154,19 @@ _getopt_internal (int argc, char **argv, const char *optstring, return result; } +/* glibc gets a LSB-compliant getopt. + Standalone applications get a POSIX-compliant getopt. */ +#if _LIBC +enum { POSIXLY_CORRECT = 0 }; +#else +enum { POSIXLY_CORRECT = 1 }; +#endif + int getopt (int argc, char *const *argv, const char *optstring) { - return _getopt_internal (argc, (char **) argv, optstring, NULL, NULL, 0, 1); + return _getopt_internal (argc, (char **) argv, optstring, NULL, NULL, 0, + POSIXLY_CORRECT); } diff --git a/lib/getopt1.c b/lib/getopt1.c index 2ceff8ec8..a7472371f 100644 --- a/lib/getopt1.c +++ b/lib/getopt1.c @@ -41,10 +41,11 @@ #endif int -getopt_long (int argc, char **argv, const char *options, +getopt_long (int argc, char *__getopt_argv_const *argv, const char *options, const struct option *long_options, int *opt_index) { - return _getopt_internal (argc, argv, options, long_options, opt_index, 0, 0); + return _getopt_internal (argc, (char **) argv, options, long_options, + opt_index, 0, 0); } int @@ -62,10 +63,12 @@ _getopt_long_r (int argc, char **argv, const char *options, instead. */ int -getopt_long_only (int argc, char **argv, const char *options, +getopt_long_only (int argc, char *__getopt_argv_const *argv, + const char *options, const struct option *long_options, int *opt_index) { - return _getopt_internal (argc, argv, options, long_options, opt_index, 1, 0); + return _getopt_internal (argc, (char **) argv, options, long_options, + opt_index, 1, 0); } int diff --git a/lib/getopt_.h b/lib/getopt_.h index c5c836e64..6eaa7eb91 100644 --- a/lib/getopt_.h +++ b/lib/getopt_.h @@ -53,6 +53,17 @@ # define optopt __GETOPT_PREFIX##optopt #endif +/* Standalone applications get correct prototypes for getopt_long and + getopt_long_only; they declare "char **argv". libc uses prototypes + with "char *const *argv" that are incorrect because getopt_long and + getopt_long_only can permute argv; this is required for backward + compatibility (e.g., for LSB 2.0.1). */ +#if defined __GETOPT_PREFIX && !defined __need_getopt +# define __getopt_argv_const /* empty */ +#else +# define __getopt_argv_const const +#endif + /* If __GNU_LIBRARY__ is not already defined, either we are being used standalone, or this is the first header included in the source file. If we are being used with glibc, we need to include <features.h>, but @@ -178,11 +189,11 @@ extern int getopt (int ___argc, char *const *___argv, const char *__shortopts) __THROW; #ifndef __need_getopt -extern int getopt_long (int ___argc, char **___argv, +extern int getopt_long (int ___argc, char *__getopt_argv_const *___argv, const char *__shortopts, const struct option *__longopts, int *__longind) __THROW; -extern int getopt_long_only (int ___argc, char **___argv, +extern int getopt_long_only (int ___argc, char *__getopt_argv_const *___argv, const char *__shortopts, const struct option *__longopts, int *__longind) __THROW; |