Module Name:    src
Committed By:   kre
Date:           Sun Oct 28 18:16:01 UTC 2018

Modified Files:
        src/bin/sh: jobs.c

Log Message:
Use strsignal() rather than direct reference to sys_siglist[]
(apart from being cleaner, it also simplifies the code, as
strsignal() never fails ... it also removes one reference to NSIG).


To generate a diff of this commit:
cvs rdiff -u -r1.101 -r1.102 src/bin/sh/jobs.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/jobs.c
diff -u src/bin/sh/jobs.c:1.101 src/bin/sh/jobs.c:1.102
--- src/bin/sh/jobs.c:1.101	Thu Sep 13 22:12:35 2018
+++ src/bin/sh/jobs.c	Sun Oct 28 18:16:01 2018
@@ -1,4 +1,4 @@
-/*	$NetBSD: jobs.c,v 1.101 2018/09/13 22:12:35 kre Exp $	*/
+/*	$NetBSD: jobs.c,v 1.102 2018/10/28 18:16:01 kre Exp $	*/
 
 /*-
  * Copyright (c) 1991, 1993
@@ -37,7 +37,7 @@
 #if 0
 static char sccsid[] = "@(#)jobs.c	8.5 (Berkeley) 5/4/95";
 #else
-__RCSID("$NetBSD: jobs.c,v 1.101 2018/09/13 22:12:35 kre Exp $");
+__RCSID("$NetBSD: jobs.c,v 1.102 2018/10/28 18:16:01 kre Exp $");
 #endif
 #endif /* not lint */
 
@@ -494,11 +494,7 @@ showjob(struct output *out, struct job *
 			else /* WIFSIGNALED(ps->status) */
 #endif
 				st = WTERMSIG(ps->status);
-			st &= 0x7f;
-			if (st < NSIG && sys_siglist[st])
-				scopyn(sys_siglist[st], s + col, 32);
-			else
-				fmtstr(s + col, 16, "Signal %d", st);
+			scopyn(strsignal(st), s + col, 32);
 			if (WCOREDUMP(ps->status)) {
 				col += strlen(s + col);
 				scopyn(" (core dumped)", s + col,  64 - col);

Reply via email to