PlaneteCasio
/
libc
1
0
Fork 0
Code Issues 6 Pull Requests Releases Wiki Activity

* libc/getopt.c (parse_long_options): Use fix from NetBSD's getopt 

to avoid false ambiguities.
This commit is contained in:
Corinna Vinschen 2009-01-13 09:58:43 +00:00
parent d9db4e43f3
commit 8c3efc4901
2 changed files with 24 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
winsup/cygwin/ChangeLog
View File

@ -1,3 +1,8 @@
2009-01-13 Corinna Vinschen <corinna@vinschen.de>
* libc/getopt.c (parse_long_options): Use fix from NetBSD's getopt
to avoid false ambiguities.
2009-01-12 Corinna Vinschen <corinna@vinschen.de>
* syscalls.cc (enum bin_status): New type.

28
winsup/cygwin/libc/getopt.c
View File

@ -170,10 +170,16 @@ parse_long_options(char * const *nargv, const char *options,
{
char *current_argv, *has_equal;
size_t current_argv_len;
int i, match;
int i, ambiguous, match;
#define IDENTICAL_INTERPRETATION(_x, _y) \
(long_options[(_x)].has_arg == long_options[(_y)].has_arg && \
long_options[(_x)].flag == long_options[(_y)].flag && \
long_options[(_x)].val == long_options[(_y)].val)
current_argv = place;
match = -1;
ambiguous = 0;
optind++;
@ -193,6 +199,7 @@ parse_long_options(char * const *nargv, const char *options,
if (strlen(long_options[i].name) == current_argv_len) {
/* exact match */
match = i;
ambiguous = 0;
break;
}
/*
@ -204,14 +211,16 @@ parse_long_options(char * const *nargv, const char *options,
if (match == -1) /* partial match */
match = i;
else {
/* ambiguous abbreviation */
if (PRINT_ERROR)
warnx(ambig, (int)current_argv_len,
current_argv);
optopt = 0;
return (BADCH);
}
else if (!IDENTICAL_INTERPRETATION(i, match))
ambiguous = 1;
}
if (ambiguous) {
/* ambiguous abbreviation */
if (PRINT_ERROR)
warnx(ambig, (int)current_argv_len,
current_argv);
optopt = 0;
return (BADCH);
}
if (match != -1) { /* option found */
if (long_options[match].has_arg == no_argument
@ -276,6 +285,7 @@ parse_long_options(char * const *nargv, const char *options,
return (0);
} else
return (long_options[match].val);
#undef IDENTICAL_INTERPRETATION
}
/*