From: Pedro Alves <palves@redhat.com>
To: gdb-patches@sourceware.org
Subject: [PATCH 0/5] dwarf2read.c: Some C++fycation
Date: Wed, 29 Mar 2017 02:25:00 -0000 [thread overview]
Message-ID: <1490754298-9455-1-git-send-email-palves@redhat.com> (raw)
While working on:
https://sourceware.org/ml/gdb-patches/2017-03/msg00444.html
after looking at:
https://sourceware.org/ml/gdb-patches/2017-03/msg00427.html
I noticed several things in gdb/dwarf2read.c that could be cleaned up.
This is the result.
This also serves as an earlier justification for adding several bits
that I'd like for another (larger) series I'm working on (e.g., the
gdb::optional changes). :-)
Tested on x86_64 Fedora 23.
Pedro Alves (5):
dwarf2read.c: Some C++fycation, use std::vector, std::unique_ptr
gdb::optional: Add observers
dwarf2read.c: Make dir_index and file_name_index strong typedefs
Make sect_offset and cu_offset strong typedefs instead of structs
dwarf2read.c: C++fy lnp_state_machine
gdb/Makefile.in | 6 +-
gdb/common/gdb_optional.h | 29 +
gdb/common/offset-type.h | 149 ++++
gdb/common/preprocessor.h | 31 +
gdb/common/traits.h | 34 +
gdb/common/underlying.h | 32 +
gdb/common/valid-expr.h | 108 +++
gdb/dwarf2expr.c | 45 +-
gdb/dwarf2expr.h | 16 +-
gdb/dwarf2loc.c | 37 +-
gdb/dwarf2read.c | 1494 +++++++++++++++++----------------
gdb/gdbtypes.h | 14 +-
gdb/unittests/offset-type-selftests.c | 178 ++++
13 files changed, 1369 insertions(+), 804 deletions(-)
create mode 100644 gdb/common/offset-type.h
create mode 100644 gdb/common/preprocessor.h
create mode 100644 gdb/common/traits.h
create mode 100644 gdb/common/underlying.h
create mode 100644 gdb/common/valid-expr.h
create mode 100644 gdb/unittests/offset-type-selftests.c
--
2.5.5
next reply other threads:[~2017-03-29 2:25 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-03-29 2:25 Pedro Alves [this message]
2017-03-29 2:25 ` [PATCH 3/5] dwarf2read.c: Make dir_index and file_name_index strong typedefs Pedro Alves
2017-03-29 15:36 ` Simon Marchi
2017-04-04 18:54 ` Pedro Alves
2017-04-04 19:06 ` Pedro Alves
2017-03-29 2:25 ` [PATCH 2/5] gdb::optional: Add observers Pedro Alves
2017-03-29 2:25 ` [PATCH 1/5] dwarf2read.c: Some C++fycation, use std::vector, std::unique_ptr Pedro Alves
[not found] ` <ef286ec115c7c1986e41dd08ecf14fcf@polymtl.ca>
2017-04-04 18:57 ` Pedro Alves
2017-03-29 2:25 ` [PATCH 5/5] dwarf2read.c: C++fy lnp_state_machine Pedro Alves
2017-03-29 2:32 ` [PATCH 4/5] Make sect_offset and cu_offset strong typedefs instead of structs Pedro Alves
2017-03-29 15:46 ` Simon Marchi
2017-04-05 15:13 ` Pedro Alves
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=1490754298-9455-1-git-send-email-palves@redhat.com \
--to=palves@redhat.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