gui.c: Remove silly warning on SIGINT.
authorAndre Noll <maan@tuebingen.mpg.de>
Fri, 18 Mar 2016 21:40:12 +0000 (22:40 +0100)
committerAndre Noll <maan@tuebingen.mpg.de>
Fri, 18 Mar 2016 21:40:12 +0000 (22:40 +0100)
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.

gui.c

diff --git a/gui.c b/gui.c
index 83337f8..dd7ffeb 100644 (file)
--- 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");