From 17568d782d96434537fe9698c5ebcb09f35c20cb Mon Sep 17 00:00:00 2001 From: Tom Tromey Date: Sun, 23 Jun 2019 14:28:11 -0600 Subject: [PATCH] Remove NULL checks before xfree A couple of spots in the TUI did a NULL check before an xfree. This isn't necessary, and most other cases were removed from gdb a while ago. gdb/ChangeLog 2019-06-25 Tom Tromey * tui/tui-source.c (tui_set_source_content): Don't check before xfree. * tui/tui-disasm.c (tui_disassemble): Don't check before xfree. --- gdb/ChangeLog | 6 ++++++ gdb/tui/tui-disasm.c | 6 ++---- gdb/tui/tui-source.c | 3 +-- 3 files changed, 9 insertions(+), 6 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 2e641249957..9898bcd5c91 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,9 @@ +2019-06-25 Tom Tromey + + * tui/tui-source.c (tui_set_source_content): Don't check before + xfree. + * tui/tui-disasm.c (tui_disassemble): Don't check before xfree. + 2019-06-25 Tom Tromey * tui/tui-winsource.h (tui_update_source_window_as_is) diff --git a/gdb/tui/tui-disasm.c b/gdb/tui/tui-disasm.c index d3d53d7c607..4899b1d03cf 100644 --- a/gdb/tui/tui-disasm.c +++ b/gdb/tui/tui-disasm.c @@ -59,10 +59,8 @@ tui_disassemble (struct gdbarch *gdbarch, struct tui_asm_line *asm_lines, /* Now construct each line. */ for (; count > 0; count--, asm_lines++) { - if (asm_lines->addr_string) - xfree (asm_lines->addr_string); - if (asm_lines->insn) - xfree (asm_lines->insn); + xfree (asm_lines->addr_string); + xfree (asm_lines->insn); print_address (gdbarch, pc, &gdb_dis_out); asm_lines->addr = pc; diff --git a/gdb/tui/tui-source.c b/gdb/tui/tui-source.c index 4f82cbab2db..889b9509fec 100644 --- a/gdb/tui/tui-source.c +++ b/gdb/tui/tui-source.c @@ -163,8 +163,7 @@ tui_set_source_content (struct symtab *s, = (struct tui_source_window_base *) TUI_SRC_WIN; const char *s_filename = symtab_to_filename_for_display (s); - if (TUI_SRC_WIN->title) - xfree (TUI_SRC_WIN->title); + xfree (TUI_SRC_WIN->title); TUI_SRC_WIN->title = xstrdup (s_filename); xfree (src->fullname); -- 2.30.2