diff options
author | Pedro Alves <palves@redhat.com> | 2017-11-06 15:36:47 +0000 |
---|---|---|
committer | Pedro Alves <palves@redhat.com> | 2017-11-06 16:13:05 +0000 |
commit | 6aa899ce25d73c397748e4df926da3fd7e5870e5 (patch) | |
tree | b11debf0beb0fdef449fd5f62c50958ecf933473 /gdb/event-top.h | |
parent | a94799ac1e430404fda5cbc181406cf5f3630108 (diff) | |
download | gdb-6aa899ce25d73c397748e4df926da3fd7e5870e5.zip gdb-6aa899ce25d73c397748e4df926da3fd7e5870e5.tar.gz gdb-6aa899ce25d73c397748e4df926da3fd7e5870e5.tar.bz2 |
Eliminate STOP_SIGNAL, use SIGTSTP directly
The STOP_SIGNAL macro was originally added for Convex Unix
(https://en.wikipedia.org/wiki/Convex_Computer).
In:
git show 7a67dd45ca1c:gdb/m-convex.h
we see:
~~~
/* Use SIGCONT rather than SIGTSTP because convex Unix occasionally
turkeys SIGTSTP. I think. */
#define STOP_SIGNAL SIGCONT
~~~
That's gdb-3.5, 1990... In gdb/ChangeLog-3.x we see:
~~~
Tue Apr 18 13:43:37 1989 Randall Smith (randy at apple-gunkies.ai.mit.edu)
Various changes involved in 1) getting gdb to work on the convex,
[...]
Made whatever signal indicates a stop configurable (via macro
STOP_SIGNAL).
(main): Setup use of above as a signal handler. Added check for
"-nw" in args already processed.
(command_line_input): SIGTSTP ==>STOP_SIGNAL.
~~~
Support for Convex Unix is long gone, and nothing else overrides
STOP_SIGNAL. So just use SIGTSTP directly, removing a little
obfuscation.
(I don't really understand why we override [1] readline's SIGTSTP
handler (only) when reading scripts (and then fail to restore it
properly, assuming SIG_DFL...), but I'll leave that for another pass.
[1] - Actually, starting with readline 6.3, readline is no longer
installing its handlers while GDB is in control...)
gdb/ChangeLog:
2017-11-06 Pedro Alves <palves@redhat.com>
* event-top.c: Check SIGTSTP instead of STOP_SIGNAL thoughout.
(async_init_signals): Adjust.
(handle_stop_sig): Rename to ...
(handle_sigtstp): ... this.
(async_stop_sig): Rename to ...
(async_sigtstp_handler): ... this, and delete STOP_SIGNAL !=
SIGTSTP path.
* event-top.h: Move signal.h include to the top. Check SIGTSTP
instead of STOP_SIGNAL thoughout.
(handle_stop_sig): Rename to ...
(handle_sigtstp): ... this.
* top.c (command_line_input): Replace STOP_SIGNAL -> SIGTSTP.
Diffstat (limited to 'gdb/event-top.h')
-rw-r--r-- | gdb/event-top.h | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/gdb/event-top.h b/gdb/event-top.h index 05241ad..d069c56 100644 --- a/gdb/event-top.h +++ b/gdb/event-top.h @@ -22,6 +22,8 @@ #ifndef EVENT_TOP_H #define EVENT_TOP_H +#include <signal.h> + struct cmd_list_element; /* Exported functions from event-top.c. @@ -36,14 +38,10 @@ extern void change_line_handler (int); extern void command_line_handler (char *rl); extern void command_handler (char *command); -/* Signal to catch ^Z typed while reading a command: SIGTSTP or SIGCONT. */ -#ifndef STOP_SIGNAL -#include <signal.h> #ifdef SIGTSTP -#define STOP_SIGNAL SIGTSTP -extern void handle_stop_sig (int sig); -#endif +extern void handle_sigtstp (int sig); #endif + extern void handle_sigint (int sig); extern void handle_sigterm (int sig); extern void async_request_quit (void *arg); |