From f2f46dfcdafe6bd32152d80b5e832d2fab4e6900 Mon Sep 17 00:00:00 2001 From: Simon Marchi Date: Tue, 2 May 2017 13:30:08 -0400 Subject: [PATCH] Use std::vector in lm_info_target Replace the two VEC fields with std::vector. gdb/ChangeLog: * solib-target.c: Include (struct lm_info_target) <~lm_info_target>: Remove. : Change type to std::vector. (library_list_start_segment, library_list_start_section, library_list_end_library, solib_target_relocate_section_addresses): Adjust. --- gdb/ChangeLog | 10 ++++++++++ gdb/solib-target.c | 45 ++++++++++++++++----------------------------- 2 files changed, 26 insertions(+), 29 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 0624f2fd718..ac22285d3fb 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,13 @@ +2017-05-02 Simon Marchi + + * solib-target.c: Include + (struct lm_info_target) <~lm_info_target>: Remove. + : Change type to + std::vector. + (library_list_start_segment, library_list_start_section, + library_list_end_library, + solib_target_relocate_section_addresses): Adjust. + 2017-05-02 Simon Marchi * gdbarch.sh (software_single_step): Change return type to diff --git a/gdb/solib-target.c b/gdb/solib-target.c index e25f6ab2524..2e9b96273d0 100644 --- a/gdb/solib-target.c +++ b/gdb/solib-target.c @@ -25,16 +25,11 @@ #include "target.h" #include "vec.h" #include "solib-target.h" +#include /* Private data for each loaded library. */ struct lm_info_target : public lm_info_base { - ~lm_info_target () - { - VEC_free (CORE_ADDR, this->segment_bases); - VEC_free (CORE_ADDR, this->section_bases); - } - /* The library's name. The name is normally kept in the struct so_list; it is only here during XML parsing. */ std::string name; @@ -44,11 +39,11 @@ struct lm_info_target : public lm_info_base /* The base addresses for each independently relocatable segment of this shared library. */ - VEC(CORE_ADDR) *segment_bases = NULL; + std::vector segment_bases; /* The base addresses for each independently allocatable, relocatable section of this shared library. */ - VEC(CORE_ADDR) *section_bases = NULL; + std::vector section_bases; /* The cached offsets for each section of this shared library, determined from SEGMENT_BASES, or SECTION_BASES. */ @@ -92,11 +87,11 @@ library_list_start_segment (struct gdb_xml_parser *parser, = (ULONGEST *) xml_find_attribute (attributes, "address")->value; CORE_ADDR address = (CORE_ADDR) *address_p; - if (last->section_bases != NULL) + if (!last->section_bases.empty ()) gdb_xml_error (parser, _("Library list with both segments and sections")); - VEC_safe_push (CORE_ADDR, last->segment_bases, address); + last->segment_bases.push_back (address); } static void @@ -110,11 +105,11 @@ library_list_start_section (struct gdb_xml_parser *parser, = (ULONGEST *) xml_find_attribute (attributes, "address")->value; CORE_ADDR address = (CORE_ADDR) *address_p; - if (last->segment_bases != NULL) + if (!last->segment_bases.empty ()) gdb_xml_error (parser, _("Library list with both segments and sections")); - VEC_safe_push (CORE_ADDR, last->section_bases, address); + last->section_bases.push_back (address); } /* Handle the start of a element. */ @@ -141,10 +136,8 @@ library_list_end_library (struct gdb_xml_parser *parser, VEC(lm_info_target_p) **list = (VEC(lm_info_target_p) **) user_data; lm_info_target *lm_info = VEC_last (lm_info_target_p, *list); - if (lm_info->segment_bases == NULL - && lm_info->section_bases == NULL) - gdb_xml_error (parser, - _("No segment or section bases defined")); + if (lm_info->segment_bases.empty () && lm_info->section_bases.empty ()) + gdb_xml_error (parser, _("No segment or section bases defined")); } @@ -350,12 +343,11 @@ solib_target_relocate_section_addresses (struct so_list *so, = ((struct section_offsets *) xzalloc (SIZEOF_N_SECTION_OFFSETS (num_sections))); - if (li->section_bases) + if (!li->section_bases.empty ()) { int i; asection *sect; - int num_section_bases - = VEC_length (CORE_ADDR, li->section_bases); + int num_section_bases = li->section_bases.size (); int num_alloc_sections = 0; for (i = 0, sect = so->abfd->sections; @@ -372,10 +364,6 @@ Could not relocate shared library \"%s\": wrong number of ALLOC sections"), { int bases_index = 0; int found_range = 0; - CORE_ADDR *section_bases; - - section_bases = VEC_address (CORE_ADDR, - li->section_bases); so->addr_low = ~(CORE_ADDR) 0; so->addr_high = 0; @@ -389,7 +377,7 @@ Could not relocate shared library \"%s\": wrong number of ALLOC sections"), { CORE_ADDR low, high; - low = section_bases[i]; + low = li->section_bases[i]; high = low + bfd_section_size (so->abfd, sect) - 1; if (low < so->addr_low) @@ -399,8 +387,7 @@ Could not relocate shared library \"%s\": wrong number of ALLOC sections"), gdb_assert (so->addr_low <= so->addr_high); found_range = 1; } - li->offsets->offsets[i] - = section_bases[bases_index]; + li->offsets->offsets[i] = li->section_bases[bases_index]; bases_index++; } if (!found_range) @@ -408,7 +395,7 @@ Could not relocate shared library \"%s\": wrong number of ALLOC sections"), gdb_assert (so->addr_low <= so->addr_high); } } - else if (li->segment_bases) + else if (!li->segment_bases.empty ()) { struct symfile_segment_data *data; @@ -423,8 +410,8 @@ Could not relocate shared library \"%s\": no segments"), so->so_name); int num_bases; CORE_ADDR *segment_bases; - num_bases = VEC_length (CORE_ADDR, li->segment_bases); - segment_bases = VEC_address (CORE_ADDR, li->segment_bases); + num_bases = li->segment_bases.size (); + segment_bases = li->segment_bases.data (); if (!symfile_map_offsets_to_segments (so->abfd, data, li->offsets, num_bases, segment_bases)) -- 2.30.2