From 7208c4024ea188e904b2aa28a79c6959e9b04a6e Mon Sep 17 00:00:00 2001 From: Peter Stephenson Date: Wed, 9 Mar 2005 17:13:59 +0000 Subject: 20955: "unset foo" should return status 0 if foo was not set --- Src/builtin.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) (limited to 'Src/builtin.c') diff --git a/Src/builtin.c b/Src/builtin.c index 6fcfabfda..724a6baec 100644 --- a/Src/builtin.c +++ b/Src/builtin.c @@ -2726,8 +2726,12 @@ bin_unset(char *name, char **argv, Options ops, int func) pm = (Param) (paramtab == realparamtab ? gethashnode2(paramtab, s) : paramtab->getnode(paramtab, s)); + /* + * Unsetting an unset variable is not an error. + * This appears to be reasonably standard behaviour. + */ if (!pm) - returnval = 1; + continue; else if ((pm->flags & PM_RESTRICTED) && isset(RESTRICTED)) { zerrnam(name, "%s: restricted", pm->nam, 0); returnval = 1; -- cgit 1.4.1