gdb: fix up a few places where a char was treated as a bool
authorAndrew Burgess <aburgess@redhat.com>
Wed, 23 Aug 2023 01:39:34 +0000 (11:09 +0930)
committerAndrew Burgess <aburgess@redhat.com>
Wed, 23 Aug 2023 13:56:24 +0000 (14:56 +0100)
Spotted a few places where a char is being treated as a bool.  The GDB
style is to use explicit comparisons, so fix things up.

There should be no user visible changes after this commit.

gdb/coffread.c
gdb/dbxread.c
gdb/machoread.c

index 7998327fdaf2637b4356142ba8c5843fc10cda78..f8e14d8ad93f1dcf496230d075abed5342b06692 100644 (file)
@@ -64,7 +64,7 @@ static const registry<objfile>::key<coff_symfile_info> coff_objfile_data_key;
 
 /* Translate an external name string into a user-visible name.  */
 #define        EXTERNAL_NAME(string, abfd) \
-  (*string && *string == bfd_get_symbol_leading_char (abfd)    \
+  (*string != '\0' && *string == bfd_get_symbol_leading_char (abfd)    \
    ? string + 1 : string)
 
 /* To be an sdb debug type, type must have at least a basic or primary
index 4c585efd192818f461cbde0c214417ef88b7ce83..958b6db6615afe028ad52626f4ca571cc4089cfb 100644 (file)
@@ -486,7 +486,7 @@ record_minimal_symbol (minimal_symbol_reader &reader,
       {
        const char *tempstring = name;
 
-       if (*tempstring
+       if (*tempstring != '\0'
            && *tempstring == bfd_get_symbol_leading_char (objfile->obfd.get ()))
          ++tempstring;
        if (is_vtable_name (tempstring))
@@ -2255,7 +2255,7 @@ read_ofile_symtab (struct objfile *objfile, legacy_psymtab *pst)
            processing_gcc_compilation = 1;
          else if (strcmp (namestring, GCC2_COMPILED_FLAG_SYMBOL) == 0)
            processing_gcc_compilation = 2;
-         if (*tempstring
+         if (*tempstring != '\0'
              && *tempstring == bfd_get_symbol_leading_char (symfile_bfd))
            ++tempstring;
          if (startswith (tempstring, "__gnu_compiled"))
index 615e08c6afa85e5091ced6eec59e5ca0db9256ad..38c252c2861d0c30bc9c14db4e935f259c318814 100644 (file)
@@ -394,7 +394,7 @@ macho_resolve_oso_sym_with_minsym (struct objfile *main_objfile, asymbol *sym)
   struct bound_minimal_symbol msym;
   const char *name = sym->name;
 
-  if (*name
+  if (*name != '\0'
       && *name == bfd_get_symbol_leading_char (main_objfile->obfd.get ()))
     ++name;
   msym = lookup_minimal_symbol (name, NULL, main_objfile);