public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
* [PATCH 00/22] Convert ui-out subsystem to C++
@ 2016-11-24 15:24 Simon Marchi
  2016-11-24 15:24 ` [PATCH 01/22] Remove unused functions and declarations Simon Marchi
                   ` (23 more replies)
  0 siblings, 24 replies; 71+ messages in thread
From: Simon Marchi @ 2016-11-24 15:24 UTC (permalink / raw)
  To: gdb-patches; +Cc: Simon Marchi

Hi all,

This patchset converts the various data structures of the ui-out susbsystem to
C++ classes.  Except when stated otherwise, the goal is to preserve the same
structure, which was already very class-like, and the same behaviour.  There
are also a few cleanup patches here and there.

The patch "Class-ify ui_out" introduces many clang -Wmismatched-tags warnings,
which John aimed to fix in another series.  I am preparing a patch that fixes
them, but since it will only be boring and mechanical changes, I didn't want to
delay the submission of this series further.

The series is also available here:

  https://github.com/simark/binutils-gdb/tree/cxx-ui-out-v1

Simon

Simon Marchi (22):
  Remove unused functions and declarations
  Rename ui_out_data to mi_ui_out_data
  Remove ui_out_destroy
  Fix return value of uo_redirect
  Constify wrap_here/wrap_hint code path
  Remove verbosity from ui_out_message and friends
  Remove stale comments
  Use new/delete instead of malloc/free-based functions
  Use std::vector for ui_out::levels
  Use std::vector for mi_ui_out_data::streams
  Use std::vector for cli_ui_out_data::streams
  Use std::string in ui_out_table
  Replace hand-made linked list of ui_out_hdr by vector and iterator
  Use std::string for ui_out_hdr's text fields
  Class-ify ui_out_hdr
  Class-ify ui_out_level
  Simplify ui-out level code
  ui_out_table: Replace boolean flag with enum
  Class-ify ui_out_impl
  Class-ify ui_out_table
  Class-ify ui_out
  Introduce enum_flag type for ui_out flags

 gdb/ada-lang.c              |  47 ++-
 gdb/ada-tasks.c             |  63 ++-
 gdb/auto-load.c             |  29 +-
 gdb/break-catch-sig.c       |  16 +-
 gdb/break-catch-syscall.c   |  40 +-
 gdb/break-catch-throw.c     |  52 ++-
 gdb/breakpoint.c            | 592 +++++++++++++--------------
 gdb/cli-out.c               | 321 +++++----------
 gdb/cli-out.h               |  77 ++--
 gdb/cli/cli-cmds.c          |   2 +-
 gdb/cli/cli-logging.c       |  12 +-
 gdb/cli/cli-script.c        |  84 ++--
 gdb/cli/cli-setshow.c       |  14 +-
 gdb/cp-abi.c                |  22 +-
 gdb/darwin-nat-info.c       |  48 +--
 gdb/disasm.c                |  51 ++-
 gdb/gdb_bfd.c               |  16 +-
 gdb/guile/scm-breakpoint.c  |   4 +-
 gdb/guile/scm-ports.c       |   2 +-
 gdb/i386-linux-tdep.c       |  25 +-
 gdb/i386-tdep.c             |  32 +-
 gdb/infcmd.c                |  32 +-
 gdb/inferior.c              |  37 +-
 gdb/infrun.c                |  94 +++--
 gdb/interps.c               |   4 +-
 gdb/linespec.c              |   2 +-
 gdb/linux-thread-db.c       |  17 +-
 gdb/mi/mi-cmd-env.c         |   9 +-
 gdb/mi/mi-cmd-file.c        |  32 +-
 gdb/mi/mi-cmd-info.c        |  12 +-
 gdb/mi/mi-cmd-stack.c       |  12 +-
 gdb/mi/mi-cmd-var.c         |  77 ++--
 gdb/mi/mi-interp.c          |  81 ++--
 gdb/mi/mi-main.c            | 149 ++++---
 gdb/mi/mi-out.c             | 351 ++++++----------
 gdb/mi/mi-out.h             |  70 +++-
 gdb/mi/mi-symbol-cmds.c     |   5 +-
 gdb/osdata.c                |  10 +-
 gdb/probe.c                 |  67 ++-
 gdb/progspace.c             |  20 +-
 gdb/python/py-breakpoint.c  |   6 +-
 gdb/python/py-framefilter.c |  88 ++--
 gdb/record-btrace.c         |  44 +-
 gdb/remote.c                |   4 +-
 gdb/skip.c                  |  44 +-
 gdb/solib.c                 |  39 +-
 gdb/source.c                |  31 +-
 gdb/spu-tdep.c              | 124 +++---
 gdb/stack.c                 |  84 ++--
 gdb/symfile.c               |  45 +-
 gdb/thread.c                | 100 ++---
 gdb/top.c                   |   2 +-
 gdb/tracepoint.c            | 146 +++----
 gdb/tui/tui-out.c           | 156 +++----
 gdb/tui/tui.h               |   2 +-
 gdb/ui-out.c                | 973 +++++++++++++++-----------------------------
 gdb/ui-out.h                | 320 +++++++--------
 gdb/utils.c                 |   8 +-
 gdb/utils.h                 |   2 +-
 59 files changed, 2151 insertions(+), 2697 deletions(-)

-- 
2.10.0

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

end of thread, other threads:[~2016-12-02 22:22 UTC | newest]

Thread overview: 71+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-11-24 15:24 [PATCH 00/22] Convert ui-out subsystem to C++ Simon Marchi
2016-11-24 15:24 ` [PATCH 01/22] Remove unused functions and declarations Simon Marchi
2016-11-24 15:24 ` [PATCH 02/22] Rename ui_out_data to mi_ui_out_data Simon Marchi
2016-11-24 15:24 ` [PATCH 06/22] Remove verbosity from ui_out_message and friends Simon Marchi
2016-11-24 15:24 ` [PATCH 09/22] Use std::vector for ui_out::levels Simon Marchi
2016-11-24 15:24 ` [PATCH 08/22] Use new/delete instead of malloc/free-based functions Simon Marchi
2016-11-24 15:24 ` [PATCH 03/22] Remove ui_out_destroy Simon Marchi
2016-11-24 15:24 ` [PATCH 07/22] Remove stale comments Simon Marchi
2016-11-24 18:38   ` Pedro Alves
2016-11-26 15:29     ` Simon Marchi
2016-11-24 15:27 ` [PATCH 10/22] Use std::vector for mi_ui_out_data::streams Simon Marchi
2016-11-24 18:38   ` Pedro Alves
2016-11-26 15:48     ` Simon Marchi
2016-11-24 15:28 ` [PATCH 12/22] Use std::string in ui_out_table Simon Marchi
2016-11-24 15:28 ` [PATCH 15/22] Class-ify ui_out_hdr Simon Marchi
2016-11-24 15:28 ` [PATCH 16/22] Class-ify ui_out_level Simon Marchi
2016-11-24 18:41   ` Pedro Alves
2016-11-26 16:22     ` Simon Marchi
2016-11-30 12:07       ` Pedro Alves
2016-11-30 12:41         ` Antoine Tremblay
2016-11-30 13:27           ` Pedro Alves
2016-11-30 13:47             ` Antoine Tremblay
2016-11-30 14:17               ` Pedro Alves
2016-11-30 14:21                 ` Antoine Tremblay
2016-11-30 20:40         ` Simon Marchi
2016-11-24 15:28 ` [PATCH 13/22] Replace hand-made linked list of ui_out_hdr by vector and iterator Simon Marchi
2016-11-24 18:41   ` Pedro Alves
2016-11-26 16:13     ` Simon Marchi
2016-12-01 20:22       ` Simon Marchi
2016-12-01 20:23         ` Pedro Alves
2016-11-24 15:28 ` [PATCH 17/22] Simplify ui-out level code Simon Marchi
2016-11-24 18:42   ` Pedro Alves
2016-11-26 16:39     ` Simon Marchi
2016-11-30 12:08       ` Pedro Alves
2016-11-24 15:28 ` [PATCH 14/22] Use std::string for ui_out_hdr's text fields Simon Marchi
2016-11-24 15:28 ` [PATCH 18/22] ui_out_table: Replace boolean flag with enum Simon Marchi
2016-11-24 18:42   ` Pedro Alves
2016-11-26 16:47     ` Simon Marchi
2016-11-30 12:10       ` Pedro Alves
2016-11-24 15:28 ` [PATCH 11/22] Use std::vector for cli_ui_out_data::streams Simon Marchi
2016-11-24 18:41   ` Pedro Alves
2016-11-26 15:51     ` Simon Marchi
2016-11-24 15:32 ` [PATCH 22/22] Introduce enum_flag type for ui_out flags Simon Marchi
2016-11-30 13:34   ` Pedro Alves
2016-11-30 21:32     ` Simon Marchi
2016-12-02 22:22     ` [pushed] " Simon Marchi
2016-11-24 15:36 ` [PATCH 05/22] Constify wrap_here/wrap_hint code path Simon Marchi
2016-11-24 15:36 ` [PATCH 04/22] Fix return value of uo_redirect Simon Marchi
2016-11-24 16:08 ` [PATCH 00/22] Convert ui-out subsystem to C++ Simon Marchi
2016-11-24 18:46   ` Pedro Alves
2016-11-24 19:15     ` Simon Marchi
2016-11-24 20:33       ` Simon Marchi
2016-11-24 18:47   ` Pedro Alves
2016-11-27  3:14     ` Simon Marchi
2016-12-01  2:51     ` Simon Marchi
2016-12-01 21:24       ` Simon Marchi
2016-11-24 19:11 ` [PATCH 20/22] Class-ify ui_out_table Simon Marchi
2016-11-30 12:29   ` Pedro Alves
2016-11-24 19:19 ` [PATCH 21/22] Class-ify ui_out Simon Marchi
2016-11-30 12:46   ` Pedro Alves
2016-11-30 21:47     ` Simon Marchi
2016-11-26 15:20 ` [PATCH 19/22] Class-ify ui_out_impl simon.marchi
2016-11-30 13:09   ` Pedro Alves
2016-11-30 22:38     ` Simon Marchi
2016-11-30 22:58       ` Pedro Alves
2016-12-01 19:04         ` Simon Marchi
2016-12-01 19:30           ` Pedro Alves
     [not found] ` <20161124153228.25177-20-simon.marchi@polymtl.ca>
2016-11-26 17:18   ` [PATCH 20/22] Class-ify ui_out_table Simon Marchi
2016-11-30 12:30     ` Pedro Alves
2016-11-30 21:48       ` [PATCH v2] " Simon Marchi
2016-11-30 23:01         ` 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).