diff options
author | Tom Tromey <tom@tromey.com> | 2016-06-06 14:18:23 -0600 |
---|---|---|
committer | Tom Tromey <tom@tromey.com> | 2016-07-14 10:35:40 -0600 |
commit | ac29888840f025448225e600d4cf99e126386878 (patch) | |
tree | 8c92ab02d4267d9a81cff5b770f24fb6f222b231 /gdb/disasm.c | |
parent | 764c99c18adcdf658b390f25940aed7713a6a769 (diff) | |
download | gdb-ac29888840f025448225e600d4cf99e126386878.zip gdb-ac29888840f025448225e600d4cf99e126386878.tar.gz gdb-ac29888840f025448225e600d4cf99e126386878.tar.bz2 |
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".
Diffstat (limited to 'gdb/disasm.c')
-rw-r--r-- | gdb/disasm.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/gdb/disasm.c b/gdb/disasm.c index f611721..bd1f760 100644 --- a/gdb/disasm.c +++ b/gdb/disasm.c @@ -737,13 +737,11 @@ do_assembly_only (struct gdbarch *gdbarch, struct ui_out *uiout, CORE_ADDR low, CORE_ADDR high, int how_many, int flags, struct ui_file *stb) { - int num_displayed = 0; struct cleanup *ui_out_chain; ui_out_chain = make_cleanup_ui_out_list_begin_end (uiout, "asm_insns"); - num_displayed = dump_insns (gdbarch, uiout, di, low, high, how_many, - flags, stb, NULL); + dump_insns (gdbarch, uiout, di, low, high, how_many, flags, stb, NULL); do_cleanups (ui_out_chain); } |