Re: [RFA] Change target_write_memory_blocks to use std::vector

Tom Tromey
2018-02-26T19:38:02Z

>>>>> "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