From d11600184f9a77b4f781f3041ddb3c5c69bdf41e Mon Sep 17 00:00:00 2001 From: Tom Tromey Date: Mon, 22 Jul 2013 17:58:51 +0000 Subject: [PATCH] fix init_cutu_and_read_dies init_cutu_and_read_dies creates an outer cleanup and uses its immediately nested inner cleanup conditionally. This seems overly complicated to me; simpler is to just deal with the outer cleanup and drop the inner one. That is what this patch implements. * dwarf2read.c (init_cutu_and_read_dies): Remove 'free_cu_cleanup'. Simplify cleanup handling. --- gdb/ChangeLog | 5 +++++ gdb/dwarf2read.c | 33 ++++++++++++++------------------- 2 files changed, 19 insertions(+), 19 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index d472569e3df..66039099c62 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,8 @@ +2013-07-22 Tom Tromey + + * dwarf2read.c (init_cutu_and_read_dies): Remove 'free_cu_cleanup'. + Simplify cleanup handling. + 2013-07-22 Tom Tromey * dwarf2read.c (dwarf_decode_line_header): Call do_cleanups diff --git a/gdb/dwarf2read.c b/gdb/dwarf2read.c index 70ab302fb31..05388166ce0 100644 --- a/gdb/dwarf2read.c +++ b/gdb/dwarf2read.c @@ -5015,7 +5015,7 @@ init_cutu_and_read_dies (struct dwarf2_per_cu_data *this_cu, struct die_info *comp_unit_die; int has_children; struct attribute *attr; - struct cleanup *cleanups, *free_cu_cleanup = NULL; + struct cleanup *cleanups; struct signatured_type *sig_type = NULL; struct dwarf2_section_info *abbrev_section; /* Non-zero if CU currently points to a DWO file and we need to @@ -5074,7 +5074,7 @@ init_cutu_and_read_dies (struct dwarf2_per_cu_data *this_cu, init_one_comp_unit (cu, this_cu); /* If an error occurs while loading, release our storage. */ - free_cu_cleanup = make_cleanup (free_heap_comp_unit, cu); + make_cleanup (free_heap_comp_unit, cu); } /* Get the header. */ @@ -5203,27 +5203,22 @@ init_cutu_and_read_dies (struct dwarf2_per_cu_data *this_cu, die_reader_func (&reader, info_ptr, comp_unit_die, has_children, data); /* Done, clean up. */ - if (free_cu_cleanup != NULL) + if (keep) { - if (keep) - { - /* We've successfully allocated this compilation unit. Let our - caller clean it up when finished with it. */ - discard_cleanups (free_cu_cleanup); + /* We've successfully allocated this compilation unit. Let our + caller clean it up when finished with it. */ + discard_cleanups (cleanups); - /* We can only discard free_cu_cleanup and all subsequent cleanups. - So we have to manually free the abbrev table. */ - dwarf2_free_abbrev_table (cu); + /* We can only discard free_cu_cleanup and all subsequent cleanups. + So we have to manually free the abbrev table. */ + dwarf2_free_abbrev_table (cu); - /* Link this CU into read_in_chain. */ - this_cu->cu->read_in_chain = dwarf2_per_objfile->read_in_chain; - dwarf2_per_objfile->read_in_chain = this_cu; - } - else - do_cleanups (free_cu_cleanup); + /* Link this CU into read_in_chain. */ + this_cu->cu->read_in_chain = dwarf2_per_objfile->read_in_chain; + dwarf2_per_objfile->read_in_chain = this_cu; } - - do_cleanups (cleanups); + else + do_cleanups (cleanups); } /* Read CU/TU THIS_CU in section SECTION, -- 2.30.2