From: Simon Marchi <simon.marchi@ericsson.com>
To: Simon Marchi <simon.marchi@polymtl.ca>, <gdb-patches@sourceware.org>
Subject: Re: [PATCH 0/3] Get rid of inferior_list
Date: Sat, 14 Oct 2017 13:12:00 -0000 [thread overview]
Message-ID: <918346a5-e9e2-5656-07ea-a6743e65d16c@ericsson.com> (raw)
In-Reply-To: <20171009143036.10215-1-simon.marchi@polymtl.ca>
On 2017-10-09 10:30 AM, Simon Marchi wrote:
> This patchset removes the inferior_list/inferior_list_entry structures,
> in favor of using std::list. When working in gdbserver, I find it a bit
> cumbersome to work with the threads and processes list, having to cast
> from inferior_list_entry to the real type.
>
> Simon Marchi (3):
> gdbserver: Use std::list for all_dlls
> gdbserver: Use std::list for all_processes
> gdbserver: use std::list for all_threads
>
> gdb/gdbserver/dll.c | 36 ++---
> gdb/gdbserver/dll.h | 8 +-
> gdb/gdbserver/gdbthread.h | 109 ++++++++++++++-
> gdb/gdbserver/inferiors.c | 301 +++++++++++------------------------------
> gdb/gdbserver/inferiors.h | 124 ++++++++---------
> gdb/gdbserver/linux-arm-low.c | 5 +-
> gdb/gdbserver/linux-low.c | 277 ++++++++++++++++---------------------
> gdb/gdbserver/linux-mips-low.c | 4 +-
> gdb/gdbserver/linux-x86-low.c | 23 ++--
> gdb/gdbserver/lynx-low.c | 10 +-
> gdb/gdbserver/regcache.c | 18 +--
> gdb/gdbserver/server.c | 197 +++++++++------------------
> gdb/gdbserver/target.c | 7 +-
> gdb/gdbserver/thread-db.c | 4 +-
> gdb/gdbserver/tracepoint.c | 8 +-
> gdb/gdbserver/win32-i386-low.c | 4 +-
> gdb/gdbserver/win32-low.c | 16 +--
> 17 files changed, 479 insertions(+), 672 deletions(-)
>
I pushed this in (including the v2 for patch 1/3).
Simon
prev parent reply other threads:[~2017-10-14 13:12 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-10-09 14:30 Simon Marchi
2017-10-09 14:30 ` [PATCH 1/3] gdbserver: Use std::list for all_dlls Simon Marchi
2017-10-09 14:44 ` Pedro Alves
2017-10-09 14:49 ` Simon Marchi
2017-10-09 15:12 ` [PATCH v2] " Simon Marchi
2017-10-09 15:33 ` Pedro Alves
2017-10-09 15:46 ` Simon Marchi
2017-10-09 14:30 ` [PATCH 2/3] gdbserver: Use std::list for all_processes Simon Marchi
2017-10-09 14:30 ` [PATCH 3/3] gdbserver: use std::list for all_threads Simon Marchi
2017-10-14 13:12 ` Simon Marchi [this message]
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=918346a5-e9e2-5656-07ea-a6743e65d16c@ericsson.com \
--to=simon.marchi@ericsson.com \
--cc=gdb-patches@sourceware.org \
--cc=simon.marchi@polymtl.ca \
/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