mbox series

[0/2] Remove some use of xfree from MI

Message ID 20230609-mi-xfree-v1-0-04e9c8d9a679@adacore.com
Headers
Series Remove some use of xfree from MI |

Message

Tom Tromey June 9, 2023, 4:46 p.m. UTC 
  This series removes some uses of xfree from the MI code.  No visible
changes are expected.  Regression tested on x86-64 Fedora 36.

---
Tom Tromey (2):
      Use std::string for MI token
      Use unique_xmalloc_ptr for mi_parse::command

 gdb/mi/mi-cmds.c      |  4 ++--
 gdb/mi/mi-main.c      | 28 ++++++++++++++--------------
 gdb/mi/mi-main.h      |  2 +-
 gdb/mi/mi-parse.c     | 30 ++++++++++++------------------
 gdb/mi/mi-parse.h     | 15 ++++++++-------
 gdb/python/py-micmd.c |  2 +-
 6 files changed, 38 insertions(+), 43 deletions(-)
---
base-commit: 05d63bafad71b534c041275831489f3e0dedcf3f
change-id: 20230609-mi-xfree-47c5b7daed76

Best regards,

Comments

Keith Seitz June 16, 2023, 6:03 p.m. UTC | #1 
On 6/9/23 09:46, Tom Tromey via Gdb-patches wrote:
> This series removes some uses of xfree from the MI code.  No visible
> changes are expected.  Regression tested on x86-64 Fedora 36.

Ha, I am also using F36 (and regtested there, too).

The changes are pretty mechanical and LGTM.

Reviewed-by: Keith Seitz <keiths@redhat.com>

Keith