diff options
author | Simon Marchi <simon.marchi@polymtl.ca> | 2017-10-14 08:43:54 -0400 |
---|---|---|
committer | Simon Marchi <simon.marchi@ericsson.com> | 2017-10-14 08:43:55 -0400 |
commit | 4cdd21a8d3fd943d6993e9d053edf09583802744 (patch) | |
tree | 23e8793e081cdccbc32fa04ae7af0b78bf4933ca /gdb/tracefile-tfile.c | |
parent | d0d292a27402ee2d3f91d541371f134f91730373 (diff) | |
download | gdb-4cdd21a8d3fd943d6993e9d053edf09583802744.zip gdb-4cdd21a8d3fd943d6993e9d053edf09583802744.tar.gz gdb-4cdd21a8d3fd943d6993e9d053edf09583802744.tar.bz2 |
Use std::vector for traceframe_info::memory
Straightforward change from a VEC to std::vector. This allows making
the destruction of a traceframe_info trivial.
I added a constructor with parameters to mem_range to be able to
emplace_back directly with the values. It is necessary to leave a
default constructor there because mem_range is still used in a VEC.
gdb/ChangeLog:
* memrange.h (struct mem_range): Add constructors.
* tracepoint.h (struct traceframe_info) <memory>: Change type to
std::vector<mem_range>.
* tracepoint.c (free_traceframe_info): Don't manually free
vector.
(traceframe_info_start_memory): Adjust to vector change.
(traceframe_available_memory): Likewise.
* tracefile-tfile.c (build_traceframe_info): Likewise.
* ctf.c (ctf_traceframe_info): Likewise.
Diffstat (limited to 'gdb/tracefile-tfile.c')
-rw-r--r-- | gdb/tracefile-tfile.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/gdb/tracefile-tfile.c b/gdb/tracefile-tfile.c index 0075581..cc77b6c 100644 --- a/gdb/tracefile-tfile.c +++ b/gdb/tracefile-tfile.c @@ -1050,7 +1050,6 @@ build_traceframe_info (char blocktype, void *data) { case 'M': { - struct mem_range *r; ULONGEST maddr; unsigned short mlen; @@ -1064,10 +1063,7 @@ build_traceframe_info (char blocktype, void *data) 2, gdbarch_byte_order (target_gdbarch ())); - r = VEC_safe_push (mem_range_s, info->memory, NULL); - - r->start = maddr; - r->length = mlen; + info->memory.emplace_back (maddr, mlen); break; } case 'V': |