Remove some variables but call functions for side effects
authorTom Tromey <tom@tromey.com>
Mon, 6 Jun 2016 20:18:23 +0000 (14:18 -0600)
committerTom Tromey <tom@tromey.com>
Thu, 14 Jul 2016 16:35:40 +0000 (10:35 -0600)
commitac29888840f025448225e600d4cf99e126386878
tree8c92ab02d4267d9a81cff5b770f24fb6f222b231
parent764c99c18adcdf658b390f25940aed7713a6a769
Remove some variables but call functions for side effects

This patch consolidates the (possibly-questionable) spots where we
remove a declaration but continue to call some function for side
effects.  In a couple of cases it wasn't entirely clear to me that
this mattered; and in some other cases it might be more aesthetically
pleasing to use ATTRIBUTE_UNUSED.  So, I broke this out into a
separate patch for simpler review.

2016-07-14  Tom Tromey  <tom@tromey.com>

* arch-utils.c (default_skip_permanent_breakpoint): Remove
"bp_insn".
* disasm.c (do_assembly_only): Remove "num_displayed".
* dwarf2read.c (read_abbrev_offset): Remove "length".
(dwarf_decode_macro_bytes) <DW_MACINFO_vendor_ext>: Remove
"constant".
* m32c-tdep.c (make_regs): Remove "r2hl", "r3hl", and "intbhl".
* microblaze-tdep.c (microblaze_frame_cache): Remove "func".
* tracefile.c (trace_save): Remove "status".
gdb/ChangeLog
gdb/arch-utils.c
gdb/disasm.c
gdb/dwarf2read.c
gdb/m32c-tdep.c
gdb/microblaze-tdep.c
gdb/tracefile.c