This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [RFA 3/3] Use std::vector in gdb_bfd_data

From: Tom Tromey <tom at tromey dot com>
To: Simon Marchi <simon dot marchi at polymtl dot ca>
Cc: Tom Tromey <tom at tromey dot com>, gdb-patches at sourceware dot org
Date: Thu, 19 Oct 2017 15:56:30 -0600
Subject: Re: [RFA 3/3] Use std::vector in gdb_bfd_data
Authentication-results: sourceware.org; auth=none
References: <20171018041449.10019-1-tom@tromey.com> <20171018041449.10019-4-tom@tromey.com> <bacd4dabf65b40e23fc67087f2258eea@polymtl.ca>

>>>>> "Simon" == Simon Marchi <simon.marchi@polymtl.ca> writes:
Simon> But instead, can we make the vector a vector of gdb_bfd_ref_ptr, so
Simon> that the default destructor would take care of everything?

Yeah, good idea.  I made this change as well.

Note this can't be done with archive_bfd, because I think the rule is
that members must be closed before the archive is closed.  Though
perhaps we could move the bfd closing into the destructor as well.

Tom

References:
- [RFA 0/3] C++-ify gdb_bfd
  - From: Tom Tromey
- [RFA 3/3] Use std::vector in gdb_bfd_data
  - From: Tom Tromey
- Re: [RFA 3/3] Use std::vector in gdb_bfd_data
  - From: Simon Marchi

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]