[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