From 82cb27ff6b4e1ae811e72a02dddc3db4c9ff97b8 Mon Sep 17 00:00:00 2001 From: Tom Tromey Date: Mon, 11 Mar 2019 10:22:17 -0600 Subject: [PATCH] Remove redundant assignment from dwarf2_find_containing_comp_unit dwarf2_find_containing_comp_unit has two assignments to "this_cu" in quick succession, both of which are just: this_cu = dwarf2_per_objfile->all_comp_units[low]; ... with no intervening assignments. This patch removes the second assignment. I'm checking this in as obvious. Tested on x86-64 Fedora 29. gdb/ChangeLog 2019-03-11 Tom Tromey * dwarf2read.c (dwarf2_find_containing_comp_unit): Remove redundant assignment to "this_cu". --- gdb/ChangeLog | 5 +++++ gdb/dwarf2read.c | 1 - 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 63b8095d993..ee9fabecf0f 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,8 @@ +2019-03-11 Tom Tromey + + * dwarf2read.c (dwarf2_find_containing_comp_unit): Remove + redundant assignment to "this_cu". + 2019-03-08 Simon Marchi * gdbtypes.c (rank_one_type): Remove unnecessary cases from switch. diff --git a/gdb/dwarf2read.c b/gdb/dwarf2read.c index 0c59dcd2042..7c07a10e8c0 100644 --- a/gdb/dwarf2read.c +++ b/gdb/dwarf2read.c @@ -25157,7 +25157,6 @@ dwarf2_find_containing_comp_unit (sect_offset sect_off, } else { - this_cu = dwarf2_per_objfile->all_comp_units[low]; if (low == dwarf2_per_objfile->all_comp_units.size () - 1 && sect_off >= this_cu->sect_off + this_cu->length) error (_("invalid dwarf2 offset %s"), sect_offset_str (sect_off)); -- 2.30.2