gdb: fix some clang-tidy readability-misleading-indentation warnings
authorSimon Marchi <simon.marchi@efficios.com>
Mon, 31 Jan 2022 14:44:46 +0000 (09:44 -0500)
committerSimon Marchi <simon.marchi@polymtl.ca>
Mon, 31 Jan 2022 17:22:47 +0000 (12:22 -0500)
I have warnings like these showing in my editor all the time, so I
thought I'd run clang-tidy with this diagnostic on all the files (that I
can compile) and fix them.

There is still one warning, in utils.c, but that's because some code
is mixed up with preprocessor macros (#ifdef TUI), so I think there no
good solution there.

Change-Id: I345175fc7dd865318f0fbe61ac026c88c3b6a96b

gdb/disasm.c
gdb/dtrace-probe.c
gdb/hppa-tdep.c
gdb/mips-tdep.c
gdb/p-valprint.c
gdb/sparc64-tdep.c
gdb/tracepoint.c

index 3000e5dddad98d9de58b42bd35167bec5f20cdc1..5cd1f5adbd226dd349f278323e1ca5174673fa31 100644 (file)
@@ -189,7 +189,7 @@ line_is_less_than (const deprecated_dis_line_entry &mle1,
     {
       if (mle1.start_pc != mle2.start_pc)
        val = mle1.start_pc < mle2.start_pc;
-    else
+      else
        val = mle1.line < mle2.line;
     }
   else
index 9aeefb6060c4a51f880ec5c67ed747712f26e262..6f01edf3924c987d9f6eb17ea875605492f59598 100644 (file)
@@ -851,7 +851,7 @@ dtrace_static_probe_ops::get_probes
          if (bfd_malloc_and_get_section (abfd, sect, &dof) && dof != NULL)
            dtrace_process_dof (sect, objfile, probesp,
                                (struct dtrace_dof_hdr *) dof);
-        else
+         else
            complaint (_("could not obtain the contents of"
                         "section '%s' in objfile `%s'."),
                       bfd_section_name (sect), bfd_get_filename (abfd));
index 32c543570458cbaf5a5ff844f4b01def4cd8b22b..7734115b744c16284c52d88c75484653d6988b8e 100644 (file)
@@ -2171,7 +2171,7 @@ hppa_frame_cache (struct frame_info *this_frame, void **this_cache)
              fprintf_unfiltered (gdb_stdlog, " (base=%s) [saved]",
                                  paddress (gdbarch, cache->base));
       }
-    else
+     else
       {
        /* The prologue has been slowly allocating stack space.  Adjust
           the SP back.  */
index e5f8c6b20533c8950a5bd36d34cfe7f34fa602da..1c080bbe50fe208831fb48694ccae05c79cdd742 100644 (file)
@@ -2063,7 +2063,7 @@ micromips_next_pc (struct regcache *regcache, CORE_ADDR pc)
          if (regcache_raw_get_signed (regcache, b0s5_reg (insn >> 16))
                != regcache_raw_get_signed (regcache, b5s5_reg (insn >> 16)))
              pc += micromips_relative_offset16 (insn);
-           else
+         else
              pc += micromips_pc_insn_size (gdbarch, pc);
          break;
 
index 635d7ed5e4de28d1b45d37c75d6398a6ce2f4a56..a88d6b9a82cd496d60ebee62a4fd9ab5c1d01d32 100644 (file)
@@ -313,7 +313,7 @@ pascal_language::value_print_inner (struct value *val,
        }
       else
        {
-          if (pascal_is_string_type (type, &length_pos, &length_size,
+         if (pascal_is_string_type (type, &length_pos, &length_size,
                                     &string_pos, &char_type, NULL) > 0)
            {
              len = extract_unsigned_integer (valaddr + length_pos,
index 95d46c83253b15e56fcf5548f762b07b26b16e80..7d9b313783a4d66eb8e329e09eb143c8380eb619 100644 (file)
@@ -327,8 +327,8 @@ adi_is_addr_mapped (CORE_ADDR vaddr, size_t cnt)
            }
        }
       }
-    else
-      warning (_("unable to open /proc file '%s'"), filename);
+  else
+    warning (_("unable to open /proc file '%s'"), filename);
 
   return false;
 }
index 95fc58fb8f066fe11b0063ea6cabc0b7d9b72a09..506af3c527ece6d3cc2722f0033416cc56a2d106 100644 (file)
@@ -2410,14 +2410,14 @@ tfind_line_command (const char *args, int from_tty)
            error (_("Cannot find a good line."));
        }
       }
-    else
-      {
-       /* Is there any case in which we get here, and have an address
-          which the user would want to see?  If we have debugging
-          symbols and no line numbers?  */
-       error (_("Line number %d is out of range for \"%s\"."),
-              sal.line, symtab_to_filename_for_display (sal.symtab));
-      }
+  else
+    {
+      /* Is there any case in which we get here, and have an address
+        which the user would want to see?  If we have debugging
+        symbols and no line numbers?  */
+      error (_("Line number %d is out of range for \"%s\"."),
+            sal.line, symtab_to_filename_for_display (sal.symtab));
+    }
 
   /* Find within range of stated line.  */
   if (args && *args)