From patchwork Thu Jan 17 14:41:35 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Tom Tromey X-Patchwork-Id: 31094 Received: (qmail 78753 invoked by alias); 17 Jan 2019 14:41:46 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Delivered-To: mailing list gdb-patches@sourceware.org Received: (qmail 78729 invoked by uid 89); 17 Jan 2019 14:41:45 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-26.9 required=5.0 tests=BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, RCVD_IN_DNSWL_NONE, SPF_HELO_PASS autolearn=ham version=3.3.2 spammy= X-HELO: gateway33.websitewelcome.com Received: from gateway33.websitewelcome.com (HELO gateway33.websitewelcome.com) (192.185.145.239) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Thu, 17 Jan 2019 14:41:41 +0000 Received: from cm17.websitewelcome.com (cm17.websitewelcome.com [100.42.49.20]) by gateway33.websitewelcome.com (Postfix) with ESMTP id C53BBB4ED5 for ; Thu, 17 Jan 2019 08:41:39 -0600 (CST) Received: from box5379.bluehost.com ([162.241.216.53]) by cmsmtp with SMTP id k8rTgN7wO90onk8rTgpdSn; Thu, 17 Jan 2019 08:41:39 -0600 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=tromey.com; s=default; h=Message-Id:Date:Subject:Cc:To:From:Sender:Reply-To:MIME-Version :Content-Type:Content-Transfer-Encoding:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: In-Reply-To:References:List-Id:List-Help:List-Unsubscribe:List-Subscribe: List-Post:List-Owner:List-Archive; bh=SVaJJXuAm3J+rF59c+pDAPPr+eBjl6TjVFSEeboFOl4=; b=vt6NvUH74fephdIrgoEU7tsO20 UdKFv3L25QRolyO4CG/ezUa45FvmcdY+7cLtzkd9xmPimo5O1moJK9t0yiHH2s0sMUjcg7Lk/Y7dY EXcbHqtBQf1H8DtDTSMUkcIkE; Received: from 75-166-72-210.hlrn.qwest.net ([75.166.72.210]:42540 helo=bapiya.Home) by box5379.bluehost.com with esmtpsa (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384:256) (Exim 4.91) (envelope-from ) id 1gk8rT-002mTq-It; Thu, 17 Jan 2019 08:41:39 -0600 From: Tom Tromey To: gdb-patches@sourceware.org Cc: Tom Tromey Subject: [PATCH] Remove cleanups from solib-target.c Date: Thu, 17 Jan 2019 07:41:35 -0700 Message-Id: <20190117144135.15959-1-tom@tromey.com> This removes the remaining cleanups from solib-target.c, plus a VEC, by using std::vector and std::unique_ptr. Tested by the buildbot. gdb/ChangeLog 2019-01-17 Tom Tromey * solib-target.c (lm_info_target_p): Remove typedef. Don't declare VEC. (solib_target_parse_libraries): Change return type. (library_list_start_segment, library_list_start_section) (library_list_end_library, library_list_start_library); Update. (solib_target_free_library_list): Remove. (solib_target_parse_libraries): Remove cleanup. Change return type. (solib_target_current_sos): Update. --- gdb/ChangeLog | 12 ++++++++ gdb/solib-target.c | 68 +++++++++++++++------------------------------- 2 files changed, 34 insertions(+), 46 deletions(-) diff --git a/gdb/solib-target.c b/gdb/solib-target.c index 972d9ef17b..51cb6ca155 100644 --- a/gdb/solib-target.c +++ b/gdb/solib-target.c @@ -50,12 +50,11 @@ struct lm_info_target : public lm_info_base section_offsets *offsets = NULL; }; -typedef lm_info_target *lm_info_target_p; -DEF_VEC_P(lm_info_target_p); +typedef std::vector> lm_info_vector; #if !defined(HAVE_LIBEXPAT) -static VEC(lm_info_target_p) * +static lm_info_vector solib_target_parse_libraries (const char *library) { static int have_warned; @@ -67,7 +66,7 @@ solib_target_parse_libraries (const char *library) "at compile time")); } - return NULL; + return lm_info_vector (); } #else /* HAVE_LIBEXPAT */ @@ -82,8 +81,8 @@ library_list_start_segment (struct gdb_xml_parser *parser, void *user_data, std::vector &attributes) { - VEC(lm_info_target_p) **list = (VEC(lm_info_target_p) **) user_data; - lm_info_target *last = VEC_last (lm_info_target_p, *list); + lm_info_vector *list = (lm_info_vector *) user_data; + lm_info_target *last = list->back ().get (); ULONGEST *address_p = (ULONGEST *) xml_find_attribute (attributes, "address")->value.get (); CORE_ADDR address = (CORE_ADDR) *address_p; @@ -101,8 +100,8 @@ library_list_start_section (struct gdb_xml_parser *parser, void *user_data, std::vector &attributes) { - VEC(lm_info_target_p) **list = (VEC(lm_info_target_p) **) user_data; - lm_info_target *last = VEC_last (lm_info_target_p, *list); + lm_info_vector *list = (lm_info_vector *) user_data; + lm_info_target *last = list->back ().get (); ULONGEST *address_p = (ULONGEST *) xml_find_attribute (attributes, "address")->value.get (); CORE_ADDR address = (CORE_ADDR) *address_p; @@ -122,12 +121,12 @@ library_list_start_library (struct gdb_xml_parser *parser, void *user_data, std::vector &attributes) { - VEC(lm_info_target_p) **list = (VEC(lm_info_target_p) **) user_data; + lm_info_vector *list = (lm_info_vector *) user_data; lm_info_target *item = new lm_info_target; item->name = (const char *) xml_find_attribute (attributes, "name")->value.get (); - VEC_safe_push (lm_info_target_p, *list, item); + list->emplace_back (item); } static void @@ -135,8 +134,8 @@ library_list_end_library (struct gdb_xml_parser *parser, const struct gdb_xml_element *element, void *user_data, const char *body_text) { - 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); + lm_info_vector *list = (lm_info_vector *) user_data; + lm_info_target *lm_info = list->back ().get (); if (lm_info->segment_bases.empty () && lm_info->section_bases.empty ()) gdb_xml_error (parser, _("No segment or section bases defined")); @@ -165,22 +164,6 @@ library_list_start_list (struct gdb_xml_parser *parser, } } -/* Discard the constructed library list. */ - -static void -solib_target_free_library_list (void *p) -{ - VEC(lm_info_target_p) **result = (VEC(lm_info_target_p) **) p; - lm_info_target *info; - int ix; - - for (ix = 0; VEC_iterate (lm_info_target_p, *result, ix, info); ix++) - delete info; - - VEC_free (lm_info_target_p, *result); - *result = NULL; -} - /* The allowed elements and attributes for an XML library list. The root element is a . */ @@ -227,23 +210,20 @@ static const struct gdb_xml_element library_list_elements[] = { { NULL, NULL, NULL, GDB_XML_EF_NONE, NULL, NULL } }; -static VEC(lm_info_target_p) * +static lm_info_vector solib_target_parse_libraries (const char *library) { - VEC(lm_info_target_p) *result = NULL; - struct cleanup *back_to = make_cleanup (solib_target_free_library_list, - &result); + lm_info_vector result; if (gdb_xml_parse_quick (_("target library list"), "library-list.dtd", library_list_elements, library, &result) == 0) { - /* Parsed successfully, keep the result. */ - discard_cleanups (back_to); + /* Parsed successfully. */ return result; } - do_cleanups (back_to); - return NULL; + result.clear (); + return result; } #endif @@ -251,9 +231,6 @@ static struct so_list * solib_target_current_sos (void) { struct so_list *new_solib, *start = NULL, *last = NULL; - VEC(lm_info_target_p) *library_list; - lm_info_target *info; - int ix; /* Fetch the list of shared libraries. */ gdb::optional library_document @@ -263,13 +240,14 @@ solib_target_current_sos (void) return NULL; /* Parse the list. */ - library_list = solib_target_parse_libraries (library_document->data ()); + lm_info_vector library_list + = solib_target_parse_libraries (library_document->data ()); - if (library_list == NULL) + if (library_list.empty ()) return NULL; /* Build a struct so_list for each entry on the list. */ - for (ix = 0; VEC_iterate (lm_info_target_p, library_list, ix, info); ix++) + for (auto &&info : library_list) { new_solib = XCNEW (struct so_list); strncpy (new_solib->so_name, info->name.c_str (), @@ -278,11 +256,12 @@ solib_target_current_sos (void) strncpy (new_solib->so_original_name, info->name.c_str (), SO_NAME_MAX_PATH_SIZE - 1); new_solib->so_original_name[SO_NAME_MAX_PATH_SIZE - 1] = '\0'; - new_solib->lm_info = info; /* We no longer need this copy of the name. */ info->name.clear (); + new_solib->lm_info = info.release (); + /* Add it to the list. */ if (!start) last = start = new_solib; @@ -293,9 +272,6 @@ solib_target_current_sos (void) } } - /* Free the library list, but not its members. */ - VEC_free (lm_info_target_p, library_list); - return start; }