diff options
| author | Denys Vlasenko | 2017-04-12 20:21:34 +0200 |
|---|---|---|
| committer | Denys Vlasenko | 2017-04-12 20:21:34 +0200 |
| commit | 352ddd3d216131757ac278e97a09ce2f4d7f53f0 (patch) | |
| tree | 6ed924aee0419bc872809811dd7462941ca21c50 /include | |
| parent | 835ad3a984c5590ae4f6c94f2f0781ea049d1ae8 (diff) | |
| download | busybox-352ddd3d216131757ac278e97a09ce2f4d7f53f0.zip busybox-352ddd3d216131757ac278e97a09ce2f4d7f53f0.tar.gz | |
Tweak GETOPT_RESET comment
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'include')
| -rw-r--r-- | include/libbb.h | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/include/libbb.h b/include/libbb.h index 11d022f..777a4a8 100644 --- a/include/libbb.h +++ b/include/libbb.h @@ -1176,19 +1176,17 @@ extern const char *applet_long_options; #endif extern uint32_t option_mask32; extern uint32_t getopt32(char **argv, const char *applet_opts, ...) FAST_FUNC; - - /* BSD-derived getopt() functions require that optind be set to 1 in * order to reset getopt() state. This used to be generally accepted * way of resetting getopt(). However, glibc's getopt() * has additional getopt() state beyond optind (specifically, glibc - * extensions ('+' and '-' at the start of the string), and requires + * extensions such as '+' and '-' at the start of the string), and requires * that optind be set to zero to reset its state. BSD-derived versions * of getopt() misbehaved if optind is set to 0 in order to reset getopt(), * and glibc's getopt() used to coredump if optind is set 1 in order * to reset getopt(). - * Then BSD introduced additional variable "optreset" which - * be set to 1 in order to reset getopt(). Sigh. Standards, anyone? + * Then BSD introduced additional variable "optreset" which should be + * set to 1 in order to reset getopt(). Sigh. Standards, anyone? * * By ~2008, OpenBSD 3.4 was changed to survive glibc-like optind = 0 * (to interpret it as if optreset was set). |