Module Name: src
Committed By: christos
Date: Wed Jan 1 18:29:39 UTC 2014
Modified Files:
src/bin/sh: parser.c
Log Message:
clarify further.
To generate a diff of this commit:
cvs rdiff -u -r1.87 -r1.88 src/bin/sh/parser.c
Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.
Modified files:
Index: src/bin/sh/parser.c
diff -u src/bin/sh/parser.c:1.87 src/bin/sh/parser.c:1.88
--- src/bin/sh/parser.c:1.87 Wed Jan 1 11:55:28 2014
+++ src/bin/sh/parser.c Wed Jan 1 13:29:39 2014
@@ -1,4 +1,4 @@
-/* $NetBSD: parser.c,v 1.87 2014/01/01 16:55:28 christos Exp $ */
+/* $NetBSD: parser.c,v 1.88 2014/01/01 18:29:39 christos Exp $ */
/*-
* Copyright (c) 1991, 1993
@@ -37,7 +37,7 @@
#if 0
static char sccsid[] = "@(#)parser.c 8.7 (Berkeley) 5/16/95";
#else
-__RCSID("$NetBSD: parser.c,v 1.87 2014/01/01 16:55:28 christos Exp $");
+__RCSID("$NetBSD: parser.c,v 1.88 2014/01/01 18:29:39 christos Exp $");
#endif
#endif /* not lint */
@@ -433,11 +433,14 @@ TRACE(("expecting DO got %s %s\n", tokna
noalias = 1;
checkkwd = 2, readtoken();
/*
- * This should be a do {} while loop as it was originally
- * since POSIX mandates at least one case statement, but
- * unfortunately both ksh and bash accept 'case x in esac'
+ * Both ksh and bash accept 'case x in esac'
* so configure scripts started taking advantage of this.
- * so do we.
+ * The page: http://pubs.opengroup.org/onlinepubs/\
+ * 009695399/utilities/xcu_chap02.html contradicts itself,
+ * as to if this is legal; the "Case Conditional Format"
+ * paragraph shows one case is required, but the "Grammar"
+ * section shows a grammar that explicitly allows the no
+ * case option.
*/
while (lasttoken != TESAC) {
*cpp = cp = (union node *)stalloc(sizeof (struct nclist));
