[11/30] Don't call clear_quit_flag after check_quit_flag

Message ID 1458328714-4938-12-git-send-email-palves@redhat.com
State New, archived
Headers

Commit Message

Pedro Alves March 18, 2016, 7:18 p.m. UTC
  Obviously not necessary since check_quit_flag clears the flag as side
effect.

gdb/ChangeLog:
yyyy-mm-dd  Pedro Alves  <palves@redhat.com>

	* remote-sim.c (gdb_os_poll_quit): Don't call clear_quit_flag.
	* remote.c (remote_wait_as): Don't call clear_quit_flag.
---
 gdb/remote-sim.c | 5 +----
 gdb/remote.c     | 5 +----
 2 files changed, 2 insertions(+), 8 deletions(-)
  

Patch

diff --git a/gdb/remote-sim.c b/gdb/remote-sim.c
index 8489eb6..11d36eb 100644
--- a/gdb/remote-sim.c
+++ b/gdb/remote-sim.c
@@ -955,10 +955,7 @@  gdb_os_poll_quit (host_callback *p)
     deprecated_ui_loop_hook (0);
 
   if (check_quit_flag ())	/* gdb's idea of quit */
-    {
-      clear_quit_flag ();	/* we've stolen it */
-      return 1;
-    }
+    return 1;
   return 0;
 }
 
diff --git a/gdb/remote.c b/gdb/remote.c
index af0a08a..d43293b 100644
--- a/gdb/remote.c
+++ b/gdb/remote.c
@@ -6935,10 +6935,7 @@  remote_wait_as (ptid_t ptid, struct target_waitstatus *status, int options)
 	  /* If the user hit C-c before this packet, or between packets,
 	     pretend that it was hit right here.  */
 	  if (check_quit_flag ())
-	    {
-	      clear_quit_flag ();
-	      sync_remote_interrupt (SIGINT);
-	    }
+	    sync_remote_interrupt (SIGINT);
 	}
 
       /* FIXME: cagney/1999-09-27: If we're in async mode we should