[binutils-gdb] gdb: bool-ify users of file_is_auto_load_safe

Simon Marchi simark@sourceware.org
Wed Jan 13 17:01:23 GMT 2021


https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=5bf7e91b2b47295b021ce3a06688d8cfb74e7fab

commit 5bf7e91b2b47295b021ce3a06688d8cfb74e7fab
Author: Simon Marchi <simon.marchi@polymtl.ca>
Date:   Wed Jan 13 12:00:37 2021 -0500

    gdb: bool-ify users of file_is_auto_load_safe
    
    A previous patch missed those.
    
    gdb/ChangeLog:
    
            * auto-load.c (auto_load_objfile_script_1): Use bool.
            (execute_script_contents): Use bool.
    
    Change-Id: I214bf7ed25af36ced375eb3ec5a403ded2fa572e

Diff:
---
 gdb/ChangeLog   |  5 +++++
 gdb/auto-load.c | 18 +++++++++---------
 2 files changed, 14 insertions(+), 9 deletions(-)

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index e9f7982cf00..58392b25cd9 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2021-01-13  Simon Marchi  <simon.marchi@polymtl.ca>
+
+	* auto-load.c (auto_load_objfile_script_1): Use bool.
+	(execute_script_contents): Use bool.
+
 2021-01-13  Simon Marchi  <simon.marchi@polymtl.ca>
 
 	* auto-load.h (auto_load_gdb_scripts_enabled): Return bool, move
diff --git a/gdb/auto-load.c b/gdb/auto-load.c
index 0d5532b4882..b6cb79af34e 100644
--- a/gdb/auto-load.c
+++ b/gdb/auto-load.c
@@ -808,10 +808,9 @@ auto_load_objfile_script_1 (struct objfile *objfile, const char *realname,
 
   if (input)
     {
-      int is_safe;
       struct auto_load_pspace_info *pspace_info;
 
-      is_safe
+      bool is_safe
 	= file_is_auto_load_safe (debugfile,
 				  _("auto-load: Loading %s script \"%s\""
 				    " by extension for objfile \"%s\".\n"),
@@ -968,7 +967,7 @@ execute_script_contents (struct auto_load_pspace_info *pspace_info,
   objfile_script_executor_func *executor;
   const char *newline, *script_text;
   const char *name;
-  int is_safe, in_hash_table;
+  int in_hash_table;
 
   /* The first line of the script is the name of the script.
      It must not contain any kind of space character.  */
@@ -1022,12 +1021,13 @@ of file %ps."),
       return;
     }
 
-  is_safe = file_is_auto_load_safe (objfile_name (objfile),
-				    _("auto-load: Loading %s script "
-				      "\"%s\" from section \"%s\" of "
-				      "objfile \"%s\".\n"),
-				    ext_lang_name (language), name,
-				    section_name, objfile_name (objfile));
+  bool is_safe
+    = file_is_auto_load_safe (objfile_name (objfile),
+			      _("auto-load: Loading %s script "
+				"\"%s\" from section \"%s\" of "
+				"objfile \"%s\".\n"),
+			      ext_lang_name (language), name,
+			      section_name, objfile_name (objfile));
 
   in_hash_table = maybe_add_script_text (pspace_info, is_safe, name, language);


More information about the Gdb-cvs mailing list