From 31120cf0451bca6e369db5bba3896a425cc4f253 Mon Sep 17 00:00:00 2001
From: jilles <jilles@FreeBSD.org>
Date: Sat, 15 Jan 2011 21:09:00 +0000
Subject: sh: Fix some things about -- in trap: * Make 'trap --' do the same as
 'trap' instead of nothing. * Make '--' stop option processing (note that '-'
 action is not an option).

Side effect: The error message for an unknown option is different.
---
 bin/sh/trap.c | 25 ++++++++++++++-----------
 1 file changed, 14 insertions(+), 11 deletions(-)

(limited to 'bin/sh')

diff --git a/bin/sh/trap.c b/bin/sh/trap.c
index 2b3a4d3..d2e5308 100644
--- a/bin/sh/trap.c
+++ b/bin/sh/trap.c
@@ -153,8 +153,18 @@ trapcmd(int argc, char **argv)
 	char *action;
 	int signo;
 	int errors = 0;
+	int i;
+
+	while ((i = nextopt("l")) != '\0') {
+		switch (i) {
+		case 'l':
+			printsignals();
+			return (0);
+		}
+	}
+	argv = argptr;
 
-	if (argc <= 1) {
+	if (*argv == NULL) {
 		for (signo = 0 ; signo < sys_nsig ; signo++) {
 			if (signo < NSIG && trap[signo] != NULL) {
 				out1str("trap -- ");
@@ -171,19 +181,12 @@ trapcmd(int argc, char **argv)
 		return 0;
 	}
 	action = NULL;
-	if (*++argv && strcmp(*argv, "--") == 0)
-		argv++;
 	if (*argv && sigstring_to_signum(*argv) == -1) {
-		if ((*argv)[0] != '-') {
-			action = *argv;
+		if (strcmp(*argv, "-") == 0)
 			argv++;
-		} else if ((*argv)[1] == '\0') {
+		else {
+			action = *argv;
 			argv++;
-		} else if ((*argv)[1] == 'l' && (*argv)[2] == '\0') {
-			printsignals();
-			return 0;
-		} else {
-			error("bad option %s", *argv);
 		}
 	}
 	while (*argv) {
-- 
cgit v1.1