svn commit: trunk/busybox/shell

vda at busybox.net vda at busybox.net
Wed Dec 3 11:59:13 UTC 2008


Author: vda
Date: 2008-12-03 03:59:12 -0800 (Wed, 03 Dec 2008)
New Revision: 24241

Log:
ash: remove TODO which seems to actually work now.
 no code changes



Modified:
   trunk/busybox/shell/ash.c


Changeset:
Modified: trunk/busybox/shell/ash.c
===================================================================
--- trunk/busybox/shell/ash.c	2008-12-03 10:46:12 UTC (rev 24240)
+++ trunk/busybox/shell/ash.c	2008-12-03 11:59:12 UTC (rev 24241)
@@ -75,7 +75,7 @@
 #endif
 
 #if defined(__uClinux__)
-#error "Do not even bother, ash will not run on uClinux"
+#error "Do not even bother, ash will not run on NOMMU machine"
 #endif
 
 
@@ -90,7 +90,7 @@
 
 #define xbarrier() do { __asm__ __volatile__ ("": : :"memory"); } while (0)
 
-/* C99 say: "char" declaration may be signed or unsigned default */
+/* C99 says: "char" declaration may be signed or unsigned by default */
 #define signed_char2int(sc) ((int)((signed char)sc))
 
 
@@ -4679,7 +4679,6 @@
 		 * $ bash -c './sleep5intoff; echo hi'
 		 * ^C^C^C^C <--- pressing ^C once a second
 		 * $ _
-		 * TODO: we do not execute "echo hi" as bash does:
 		 * $ bash -c './sleep5intoff; echo hi'
 		 * ^\^\^\^\hi <--- pressing ^\ (SIGQUIT)
 		 * $ _
@@ -5556,9 +5555,8 @@
 	result->buf = NULL;
 	result->nleft = 0;
 	result->jp = NULL;
-	if (n == NULL) {
+	if (n == NULL)
 		goto out;
-	}
 
 	saveherefd = herefd;
 	herefd = -1;
@@ -5665,7 +5663,7 @@
 	int flag;
 	int len;
 
-	/*      ifsfree(); */
+	/* ifsfree(); */
 
 	/*
 	 * This routine is slightly over-complicated for
@@ -5983,8 +5981,9 @@
 		if (*end == CTLENDVAR) {
 			if (varflags & VSNUL)
 				tail = " or null";
-		} else
+		} else {
 			msg = umsg;
+		}
 	}
 	ash_msg_and_raise_error("%.*s: %s%s", end - var - 1, var, msg, tail);
 }
@@ -6178,8 +6177,9 @@
 					idx++;
 					rmesc++;
 				}
-			} else
+			} else {
 				idx = loc;
+			}
 
 			for (loc = repl; *loc; loc++) {
 				restart_detect = stackblock();




More information about the busybox-cvs mailing list