diff options
author | Simon Marchi <simon.marchi@ericsson.com> | 2018-06-09 22:30:34 -0400 |
---|---|---|
committer | Simon Marchi <simon.marchi@polymtl.ca> | 2018-06-09 22:30:42 -0400 |
commit | 835dcf92618e294d3c6c2bedd94af712809c96a3 (patch) | |
tree | 4c93c85f195721f4d89dd4d85b492d2670a4fe2a /gdb/ChangeLog | |
parent | aac0d564cea04b1c5f386e8cea924ca59057e8b4 (diff) | |
download | gdb-835dcf92618e294d3c6c2bedd94af712809c96a3.zip gdb-835dcf92618e294d3c6c2bedd94af712809c96a3.tar.gz gdb-835dcf92618e294d3c6c2bedd94af712809c96a3.tar.bz2 |
Use std::unique_ptr in reg_buffer
Using std::unique_ptr allows to remove the manual xfree in the
destructor.
If I understand correctly, using the () after the new operator will make
sure the allocated objects will be value initialized, which for scalars
means they are zero-initialized. So it should have the same behavior as
XCNEWVEC.
gdb/ChangeLog:
* regcache.h (reg_buffer) <~reg_buffer>: Use default destructor.
<m_registers, m_register_status>: Change type to
std::unique_ptr.
* regcache.c (reg_buffer::reg_buffer): Use new instead of
XCNEWVEC.
Diffstat (limited to 'gdb/ChangeLog')
-rw-r--r-- | gdb/ChangeLog | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index c475cd9..1600232 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,5 +1,13 @@ 2018-06-09 Simon Marchi <simon.marchi@ericsson.com> + * regcache.h (reg_buffer) <~reg_buffer>: Use default destructor. + <m_registers, m_register_status>: Change type to + std::unique_ptr. + * regcache.c (reg_buffer::reg_buffer): Use new instead of + XCNEWVEC. + +2018-06-09 Simon Marchi <simon.marchi@ericsson.com> + * common/common-regcache.h (enum register_status): Add underlying type "signed char". * regcache.h (reg_buffer) <m_register_status>: Change type to |