summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--ChangeLog5
-rw-r--r--Src/builtin.c10
2 files changed, 14 insertions, 1 deletions
diff --git a/ChangeLog b/ChangeLog
index 5fc46a72c..58d30e095 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2002-09-03 Peter Stephenson <pws@csr.com>
+
+ * 17608: Src/builtins.c: comment on use and abuse of
+ BINF_SKIPINVALID.
+
2002-09-03 Oliver Kiddle <opk@zsh.org>
* Thomas Köhler: 176xx: Completion/Unix/Command/_ssh: also pass
diff --git a/Src/builtin.c b/Src/builtin.c
index 628f23f83..c3951c77f 100644
--- a/Src/builtin.c
+++ b/Src/builtin.c
@@ -303,7 +303,15 @@ execbuiltin(LinkList args, Builtin bn)
argv++;
break;
}
- /* unrecognised options to echo etc. are not really options */
+ /*
+ * Unrecognised options to echo etc. are not really
+ * options.
+ *
+ * Note this flag is not smart enough to handle option
+ * arguments. In fact, ideally it shouldn't be added
+ * to any new builtins, to preserve standard option
+ * handling as much as possible.
+ */
if (flags & BINF_SKIPINVALID) {
char *p = arg;
if (optstr)