[3/7] gdb: change type of quit_flag to bool

Message ID 20240423132517.2625632-4-simon.marchi@polymtl.ca
State New
Headers
Series More cleanup of defs.h |

Checks

Context Check Description
linaro-tcwg-bot/tcwg_gdb_build--master-arm fail Patch failed to apply
linaro-tcwg-bot/tcwg_gdb_build--master-aarch64 fail Patch failed to apply

Commit Message

Simon Marchi April 23, 2024, 1:22 p.m. UTC
  Change-Id: I7dc5189ee172e82ef5b2c4a739c011f43a84258b
---
 gdb/event-top.c | 4 ++--
 gdb/extension.c | 6 +++---
 2 files changed, 5 insertions(+), 5 deletions(-)
  

Patch

diff --git a/gdb/event-top.c b/gdb/event-top.c
index 6a2a75fe3dc8..8411ec5028a5 100644
--- a/gdb/event-top.c
+++ b/gdb/event-top.c
@@ -1113,7 +1113,7 @@  handle_sigint (int sig)
 
   /* We could be running in a loop reading in symfiles or something so
      it may be quite a while before we get back to the event loop.  So
-     set quit_flag to 1 here.  Then if QUIT is called before we get to
+     set quit_flag to true here.  Then if QUIT is called before we get to
      the event loop, we will unwind as expected.  */
   set_quit_flag ();
 
@@ -1193,7 +1193,7 @@  handle_sigterm (int sig)
 void
 async_request_quit (gdb_client_data arg)
 {
-  /* If the quit_flag has gotten reset back to 0 by the time we get
+  /* If the quit_flag has gotten reset back to false by the time we get
      back here, that means that an exception was thrown to unwind the
      current command before we got back to the event loop.  So there
      is no reason to call quit again here.  */
diff --git a/gdb/extension.c b/gdb/extension.c
index 82d37fb09aea..99e7190d80b6 100644
--- a/gdb/extension.c
+++ b/gdb/extension.c
@@ -663,7 +663,7 @@  static std::recursive_mutex ext_lang_mutex;
 /* This flag tracks quit requests when we haven't called out to an
    extension language.  it also holds quit requests when we transition to
    an extension language that doesn't have cooperative SIGINT handling.  */
-static int quit_flag;
+static bool quit_flag;
 
 /* The current extension language we've called out to, or
    extension_language_gdb if there isn't one.
@@ -877,7 +877,7 @@  set_quit_flag ()
     active_ext_lang->ops->set_quit_flag (active_ext_lang);
   else
     {
-      quit_flag = 1;
+      quit_flag = true;
 
       /* Now wake up the event loop, or any interruptible_select.  Do
 	 this after setting the flag, because signals on Windows
@@ -914,7 +914,7 @@  check_quit_flag ()
 	 interruptible_select.  The caller handles the quit
 	 request.  */
       quit_serial_event_clear ();
-      quit_flag = 0;
+      quit_flag = false;
       result = true;
     }