[RFA] Change target_write_memory_blocks to use std::vector

Tom Tromey tom@tromey.com
Mon Feb 26 19:38:00 GMT 2018


>>>>> "Simon" == Simon Marchi <simark@simark.ca> writes:

>> +  load_section_data ()
>> +  {
>> +  }

Simon> Is this empty constructor needed?

I only added it for clarity.  Is there some standard approach to this?
Or a gdb standard?

Simon> Actually, I think it would be nice to give constructors to the data
Simon> structures when possible, to make it less likely to have them in
Simon> invalid states.

Simon> Here's an example, you can integrate it in your patch if you like it.

Yeah, this seems better to me.
I will pull it in.

Tom



More information about the Gdb-patches mailing list