From c8d27334a0921d6ca76862e9c7befe12a4df6dc6 Mon Sep 17 00:00:00 2001 From: Denis Vlasenko Date: Mon, 6 Apr 2009 10:47:21 +0000 Subject: [PATCH] hush: update TODO list; + my usual pointless tweaks :( --- shell/hush.c | 43 ++++++++++++++++++++++--------------------- 1 file changed, 22 insertions(+), 21 deletions(-) diff --git a/shell/hush.c b/shell/hush.c index 04853137b..119518bd1 100644 --- a/shell/hush.c +++ b/shell/hush.c @@ -6,6 +6,7 @@ * incidentally is a good match to today's BusyBox. * * Copyright (C) 2000,2001 Larry Doolittle + * Copyright (C) 2008,2009 Denys Vlasenko * * Credits: * The parser routines proper are all original material, first @@ -44,23 +45,24 @@ * Tilde Expansion * Parameter Expansion for substring processing ${var#word} ${var%word} * - * Bash stuff maybe optional enable: + * Bash stuff (maybe optionally enable?): * &> and >& redirection of stdout+stderr * Brace expansion * reserved words: [[ ]] function select * substrings ${var:1:5} * - * Major bugs: - * job handling woefully incomplete and buggy (improved --vda) - * to-do: - * port selected bugfixes from post-0.49 busybox lash - done? + * TODOs: + * grep for "TODO" and fix (some of them are easy) * change { and } from special chars to reserved words - * builtins: return, trap, ulimit - * test magic exec with redirection only + * builtins: return, ulimit * follow IFS rules more precisely, including update semantics * figure out what to do with backslash-newline - * propagate syntax errors, die on resource errors? * continuation lines, both explicit and implicit - done? + * SIGHUP handling + * ^Z handling (and explain it in comments for mere humans) + * separate job control from interactiveness + * (testcase: booting with init=/bin/hush does not show prompt (2009-04)) + * functions * * Licensed under the GPL v2 or later, see the file LICENSE in this tarball. */ @@ -5345,9 +5347,10 @@ static int builtin_trap(char **argv) if (!G.traps) G.traps = xzalloc(sizeof(G.traps[0]) * NSIG); - if (!argv[1]) { - /* No args: print all trapped. This isn't 100% correct as we should - * be escaping the cmd so that it can be pasted back in ... + argv++; + if (!*argv) { + /* No args: print all trapped. This isn't 100% correct as we + * should be escaping the cmd so that it can be pasted back in */ for (i = 0; i < NSIG; ++i) if (G.traps[i]) @@ -5357,8 +5360,8 @@ static int builtin_trap(char **argv) new_cmd = NULL; i = 0; - /* if first arg is decimal: reset all specified */ - sig = bb_strtou(*++argv, NULL, 10); + /* If first arg is decimal: reset all specified signals */ + sig = bb_strtou(*argv, NULL, 10); if (errno == 0) { int ret; set_all: @@ -5367,7 +5370,7 @@ static int builtin_trap(char **argv) sig = get_signum(*argv++); if (sig < 0 || sig >= NSIG) { ret = EXIT_FAILURE; - /* mimic bash message exactly */ + /* Mimic bash message exactly */ bb_perror_msg("trap: %s: invalid signal specification", argv[i]); continue; } @@ -5385,7 +5388,7 @@ static int builtin_trap(char **argv) if (new_cmd) { sigaddset(&G.blocked_set, sig); } else { - /* there was a trap handler, we are removing it + /* There was a trap handler, we are removing it * (if sig has non-DFL handling, * we don't need to do anything) */ if (sig < 32 && (G.non_DFL_mask & (1 << sig))) @@ -5397,16 +5400,14 @@ static int builtin_trap(char **argv) return ret; } - /* first arg is "-": reset all specified to default */ - /* first arg is "": ignore all specified */ - /* everything else: execute first arg upon signal */ + /* First arg is "-": reset all specified to default */ + /* First arg is "": ignore all specified */ + /* Everything else: execute first arg upon signal */ if (!argv[1]) { bb_error_msg("trap: invalid arguments"); return EXIT_FAILURE; } - if (LONE_DASH(*argv)) - /* nothing! */; - else + if (NOT_LONE_DASH(*argv)) new_cmd = *argv; argv++; goto set_all; -- 2.25.1