From: Andre Noll <maan@tuebingen.mpg.de>
Date: Fri, 18 Mar 2016 21:40:12 +0000 (+0100)
Subject: gui.c: Remove silly warning on SIGINT.
X-Git-Tag: v0.5.6~23^2~3
X-Git-Url: http://git.tue.mpg.de/?a=commitdiff_plain;h=6cb6634c3498101bb28291dec1dbac7135f732d9;p=paraslash.git

gui.c: Remove silly warning on SIGINT.

Sending SIGINT to para_gui has no effect, yet we print a strange
"reset" warning in this case. This patch removes the warning and the
outdated comment which referred to do_select(), a function which was
removed long ago.

Note that (a) we still catch SIGINT, and (b) hitting CTRL+C still
causes the status task to respawn the stat process because CTRL+C
sends SIGINT to the foreground process group, which includes the
stat process.
---

diff --git a/gui.c b/gui.c
index 83337f86..dd7ffeb4 100644
--- a/gui.c
+++ b/gui.c
@@ -937,10 +937,6 @@ static int signal_post_select(struct sched *s, __a_unused void *context)
 		die(EXIT_FAILURE, "only the good die young (caught SIGTERM)\n");
 		return 1;
 	case SIGINT:
-		PARA_WARNING_LOG("caught SIGINT, reset\n");
-		/* Nothing to do. SIGINT killed our child which gets noticed
-		 * by do_select and resets everything.
-		 */
 		return 1;
 	case SIGUSR1:
 		PARA_NOTICE_LOG("got SIGUSR1, rereading configuration\n");