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] Remove most cleanups from linux-thread-db.c


>>>>> "Simon" == Simon Marchi <simon.marchi@polymtl.ca> writes:

Simon> On 2018-02-22 12:02, Tom Tromey wrote:
>> The remaining cleanup in linux-thread-db.c uses
>> make_cleanup_free_char_ptr_vec, which requires a somewhat bigger
>> change.

Simon> I had a patch for that lying around, so I posted it along with other
Simon> cleanup patches to get rid of free_char_ptr_vec here, if you want to
Simon> take a look:

Simon> https://sourceware.org/ml/gdb-patches/2018-02/msg00355.html

I skimmed this series and it looked good to me.
I definitely like the idea and I thought the particular decisions you
made, about when to use string or unique_xmalloc_ptr, made sense.

Tom


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