public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
* [RFA 0/8] C++-ification series #5
@ 2016-11-29  5:06 Tom Tromey
  2016-11-29  5:06 ` [RFA 7/8] Use unique_xmalloc_ptr in execute_gdb_command Tom Tromey
                   ` (7 more replies)
  0 siblings, 8 replies; 50+ messages in thread
From: Tom Tromey @ 2016-11-29  5:06 UTC (permalink / raw)
  To: gdb-patches

This series holds a few more C++-ification changes:

* Generalizes gdbpy_ref to an "externally refcounted" smart pointer,
  with the behavior controlled by a policy class.  This is then used
  to convert BFD refcounting.

* Adds a class to optionally unlink a file in the destructor.

* Adds a class to call value_free_to_mark in the destructor.

* Adds a class to optionally discard psymtabs in the destructor.

* Fix up one more cleanup in the Python code that I missed.

* Change demangle_parse_info to have a constructor and destructor and
  use unique_ptr for managing these.

gdb is getting relatively close to no cleanups in the Python layer
after this series (25 hits for "make_cleanup").

Please review.  I'm running this through the buildbot and will comment
when that is done.

Note that the BFD refcounting change is a bit tricky.  It probably
should have extra testing -- I tested it locally, and there's the
buildbot, and additionally I built it locally using the mingw32 cross
toolchain; but even so it's possible that there are paths that weren't
updated.  These will manifest as build failures on some other
host.

One thing that would be nice is if there were a way to build gdb such
that all the nat* code were built.  This would at least catch compiler
errors.  I think it could be done by having stub implementations of
any platform-specific APIs that are used (just a #define for each
would probably suffice), and by arranging for only the real nat*.c
file's initialization function to be called at startup.

Tom

^ permalink raw reply	[flat|nested] 50+ messages in thread

end of thread, other threads:[~2017-01-10 19:22 UTC | newest]

Thread overview: 50+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-11-29  5:06 [RFA 0/8] C++-ification series #5 Tom Tromey
2016-11-29  5:06 ` [RFA 7/8] Use unique_xmalloc_ptr in execute_gdb_command Tom Tromey
2016-11-29  5:22   ` Tom Tromey
2016-12-15  3:49     ` Tom Tromey
2016-12-20 17:48       ` Pedro Alves
2016-12-20 18:13         ` Tom Tromey
2016-12-23 20:01           ` Tom Tromey
2017-01-10 17:59             ` Pedro Alves
2017-01-10 19:22               ` Tom Tromey
2016-12-20 23:31         ` Tom Tromey
2016-12-20 23:56           ` Pedro Alves
2016-12-22 14:50             ` Tom Tromey
2016-12-22 15:09               ` Pedro Alves
2016-12-22 15:29                 ` Tom Tromey
2016-12-22 15:40                   ` Pedro Alves
2016-12-02 14:49   ` Pedro Alves
2016-12-13 13:30     ` Tom Tromey
2016-11-29  5:06 ` [RFA 8/8] Add constructor and destructor to demangle_parse_info Tom Tromey
2016-12-02 15:04   ` Pedro Alves
2016-12-13 13:50     ` Tom Tromey
2016-11-29  5:06 ` [RFA 4/8] Remove make_cleanup_discard_psymtabs Tom Tromey
2016-12-02 14:21   ` Pedro Alves
2016-11-29  5:06 ` [RFA 3/8] Introduce and use gdb::unlinker Tom Tromey
2016-12-02 13:17   ` Pedro Alves
2016-11-29  5:06 ` [RFA 1/8] Add gdb_ref_ptr.h Tom Tromey
2016-12-02 13:08   ` Pedro Alves
2016-12-02 17:46     ` Tom Tromey
2016-12-02 18:11       ` Pedro Alves
2016-12-02 19:52         ` Tom Tromey
2016-12-02 23:45           ` Pedro Alves
2016-12-03  0:05   ` Pedro Alves
2016-12-13 13:13     ` Tom Tromey
2016-11-29  5:06 ` [RFA 2/8] Use class to manage BFD reference counts Tom Tromey
2016-12-02 13:05   ` Pedro Alves
2016-12-13 13:26     ` Tom Tromey
2016-12-15  4:12       ` Tom Tromey
2016-12-20 18:18         ` Pedro Alves
2016-12-20 17:19       ` [pushed] gdb: Constify solib_find (Re: [RFA 2/8] Use class to manage BFD reference counts) Pedro Alves
2016-12-20 18:05         ` Tom Tromey
2016-11-29  5:06 ` [RFA 5/8] Add value_freer Tom Tromey
2016-12-02 14:24   ` Pedro Alves
2016-11-29  5:06 ` [RFA 6/8] Use value_freer in dwarf2_evaluate_loc_desc_full Tom Tromey
2016-12-02 14:45   ` Pedro Alves
2016-12-13 13:29     ` Tom Tromey
2016-12-20 14:49       ` Pedro Alves
2016-12-23 19:05         ` Tom Tromey
2016-12-23 19:59           ` Tom Tromey
2017-01-10 17:57             ` Pedro Alves
2016-12-23 19:59           ` Tom Tromey
2017-01-10 17:58             ` Pedro Alves

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).