From: Simon Marchi <simon.marchi@ericsson.com>
To: <gdb-patches@sourceware.org>
Cc: Simon Marchi <simon.marchi@ericsson.com>
Subject: [PATCH 0/4] Poison XNEW and friends for non-POD types
Date: Wed, 22 Nov 2017 16:41:00 -0000 [thread overview]
Message-ID: <1511368867-19365-1-git-send-email-simon.marchi@ericsson.com> (raw)
As we convert types to C++, one easy mistake to do is forget to change a
XNEW(T) to new T. This patch helps ensure that we don't use XNEW (or similar)
with types that require using new. xfree is also "poisoned" the same way.
The first three patches fix the last things that were in the way of doing this
change, and patch 4 adds the actual poisoning.
Simon Marchi (4):
Create private_inferior class hierarchy
remote: C++ify thread_item and threads_listing_context
Create private_thread_info hierarchy
Poison XNEW and friends for types that should use new/delete
gdb/Makefile.in | 2 +
gdb/aix-thread.c | 49 +++---
gdb/common/common-utils.c | 7 -
gdb/common/common-utils.h | 14 +-
gdb/common/poison.h | 132 +++++++++++++++
gdb/common/rsp-low.c | 13 ++
gdb/common/rsp-low.h | 6 +
gdb/darwin-nat-info.c | 50 +++---
gdb/darwin-nat.c | 297 ++++++++++++++++++----------------
gdb/darwin-nat.h | 35 ++--
gdb/gdbthread.h | 18 ++-
gdb/inferior.c | 4 +-
gdb/inferior.h | 9 +-
gdb/linux-thread-db.c | 41 ++---
gdb/nto-procfs.c | 30 +---
gdb/nto-tdep.c | 10 +-
gdb/nto-tdep.h | 10 +-
gdb/remote.c | 329 ++++++++++++++++----------------------
gdb/thread.c | 14 +-
gdb/unittests/rsp-low-selftests.c | 59 +++++++
20 files changed, 654 insertions(+), 475 deletions(-)
create mode 100644 gdb/unittests/rsp-low-selftests.c
--
2.7.4
next reply other threads:[~2017-11-22 16:41 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-11-22 16:41 Simon Marchi [this message]
2017-11-22 16:41 ` [PATCH 1/4] Create private_inferior class hierarchy Simon Marchi
2017-11-22 22:20 ` Joel Brobecker
2017-11-24 15:46 ` Simon Marchi
2017-11-24 21:31 ` Joel Brobecker
2017-11-23 14:09 ` Pedro Alves
2017-11-23 16:17 ` Simon Marchi
2017-11-22 16:42 ` [PATCH 4/4] Poison XNEW and friends for types that should use new/delete Simon Marchi
2017-11-23 15:02 ` Pedro Alves
2017-11-23 17:27 ` Simon Marchi
2017-11-23 17:31 ` Pedro Alves
2017-11-22 16:42 ` [PATCH 2/4] remote: C++ify thread_item and threads_listing_context Simon Marchi
2017-11-23 14:22 ` Pedro Alves
2017-11-23 16:48 ` Simon Marchi
2017-11-23 16:52 ` Pedro Alves
2017-11-22 16:42 ` [PATCH 3/4] Create private_thread_info hierarchy Simon Marchi
2017-11-23 14:41 ` Pedro Alves
2017-11-23 16:54 ` Simon Marchi
2017-11-23 16:56 ` Simon Marchi
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1511368867-19365-1-git-send-email-simon.marchi@ericsson.com \
--to=simon.marchi@ericsson.com \
--cc=gdb-patches@sourceware.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox