Garbage collect remote.c:remote_async_terminal_ours_p

remote.c:remote_async_terminal_ours_p stopped being useful after
048094accc ("target remote: Don't rely on immediate_quit (introduce
quit handlers)") and commit 41fd2b0f5d ("Make input_fd be per UI"),
which turned remote's terminal_inferior/ours methods into nops.

gdb/ChangeLog:
2017-10-26  Pedro Alves  <palves@redhat.com>

	* remote.c (remote_async_terminal_ours_p): Delete.
	(remote_open_1, remote_terminal_inferior, remote_terminal_ours):
	Remove references to 'remote_async_terminal_ours_p'.
This commit is contained in:
Pedro Alves 2017-10-26 19:53:03 +01:00
parent 514f60231c
commit 23cc4e1b28
2 changed files with 6 additions and 21 deletions

View File

@ -1,3 +1,9 @@
2017-10-26 Pedro Alves <palves@redhat.com>
* remote.c (remote_async_terminal_ours_p): Delete.
(remote_open_1, remote_terminal_inferior, remote_terminal_ours):
Remove references to 'remote_async_terminal_ours_p'.
2017-10-26 Yao Qi <yao.qi@linaro.org> 2017-10-26 Yao Qi <yao.qi@linaro.org>
* breakpoint.h (regular_breakpoint_inserted_here_p): Remove. * breakpoint.h (regular_breakpoint_inserted_here_p): Remove.

View File

@ -1007,11 +1007,6 @@ show_remotebreak (struct ui_file *file, int from_tty,
static unsigned int remote_address_size; static unsigned int remote_address_size;
/* Temporary to track who currently owns the terminal. See
remote_terminal_* for more details. */
static int remote_async_terminal_ours_p;
/* User configurable variables for the number of characters in a /* User configurable variables for the number of characters in a
memory read/write packet. MIN (rsa->remote_packet_size, memory read/write packet. MIN (rsa->remote_packet_size,
@ -5117,9 +5112,6 @@ remote_open_1 (const char *name, int from_tty,
readahead_cache_invalidate (); readahead_cache_invalidate ();
/* Start out by owning the terminal. */
remote_async_terminal_ours_p = 1;
if (target_async_permitted) if (target_async_permitted)
{ {
/* FIXME: cagney/1999-09-23: During the initial connection it is /* FIXME: cagney/1999-09-23: During the initial connection it is
@ -6303,15 +6295,6 @@ interrupt_query (void)
static void static void
remote_terminal_inferior (struct target_ops *self) remote_terminal_inferior (struct target_ops *self)
{ {
/* FIXME: cagney/1999-09-27: Make calls to target_terminal::*()
idempotent. The event-loop GDB talking to an asynchronous target
with a synchronous command calls this function from both
event-top.c and infrun.c/infcmd.c. Once GDB stops trying to
transfer the terminal to the target when it shouldn't this guard
can go away. */
if (!remote_async_terminal_ours_p)
return;
remote_async_terminal_ours_p = 0;
/* NOTE: At this point we could also register our selves as the /* NOTE: At this point we could also register our selves as the
recipient of all input. Any characters typed could then be recipient of all input. Any characters typed could then be
passed on down to the target. */ passed on down to the target. */
@ -6320,10 +6303,6 @@ remote_terminal_inferior (struct target_ops *self)
static void static void
remote_terminal_ours (struct target_ops *self) remote_terminal_ours (struct target_ops *self)
{ {
/* See FIXME in remote_terminal_inferior. */
if (remote_async_terminal_ours_p)
return;
remote_async_terminal_ours_p = 1;
} }
static void static void