From: Philipp Rudo <prudo@linux.vnet.ibm.com>
To: gdb-patches@sourceware.org
Cc: arnez@linux.vnet.ibm.com
Subject: [PATCH 00/11] Cleanup deprecated_*_hooks in defs.h
Date: Tue, 28 Feb 2017 14:50:00 -0000 [thread overview]
Message-ID: <20170228145016.22604-1-prudo@linux.vnet.ibm.com> (raw)
Hi everybody
this series cleans up several hooks in defs.h which were deprecated in 2004
(9a4105ab). Most of these hooks are no longer used and can be removed without
problem.
The last patch of this series contains a minor cange for TUI. According to
the comment in tui/tui-hooks.c:tui_install_hooks the hook is only used to
check if TUI is active. Thus directly checking tui_active should do the
trick. A quick test does not show any change of behavior. Nevertheless it
would be better if somebody with more experience on TUI takes a closer look.
After this series only two deprecated hooks remain in defs.h
(deprecated_show_load_progress and deprecated_query_hook). Both of them are
still used in the MI interpreter. It would be nice if somebody with
experience on MI could take a look at them.
Philipp
Philipp Rudo (11):
Remove unused deprecated_{pre|post}_add_symbol_hook
Remove unused deprecated_ui_loop_hook
Remove unused deprecated_warning_hook
Remove unused deprecated_interactive_hook
Remove unused deprecated_readline{_begin|_end}_hook
Remove unused deprecated_context_hook
Remove unused deprecated_target_wait_hook
Remove unused deprecated_{attach|detach}_hook
Remove unused deprecated_call_command_hook
Remove unused deprecated_ui_load_progress_hook
Remove deprecated_print_frame_info_listing_hook
gdb/cli/cli-script.c | 21 +++------------
gdb/complaints.c | 2 --
gdb/defs.h | 26 -------------------
gdb/infcmd.c | 7 -----
gdb/infrun.c | 26 +++++--------------
gdb/interps.c | 9 -------
gdb/remote-sim.c | 3 ---
gdb/ser-base.c | 13 ----------
gdb/ser-tcp.c | 11 +-------
gdb/ser-unix.c | 15 -----------
gdb/serial.h | 5 +---
gdb/stack.c | 42 +++++++++++++-----------------
gdb/symfile.c | 25 ++++--------------
gdb/target.c | 3 ---
gdb/top.c | 72 +---------------------------------------------------
gdb/tui/tui-hooks.c | 19 --------------
gdb/utils.c | 36 ++++++++++----------------
gdb/windows-nat.c | 10 --------
18 files changed, 48 insertions(+), 297 deletions(-)
--
2.8.4
next reply other threads:[~2017-02-28 14:50 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-02-28 14:50 Philipp Rudo [this message]
2017-02-28 14:50 ` [PATCH 03/11] Remove unused deprecated_warning_hook Philipp Rudo
2017-02-28 14:50 ` [PATCH 10/11] Remove unused deprecated_ui_load_progress_hook Philipp Rudo
2017-02-28 14:50 ` [PATCH 08/11] Remove unused deprecated_{attach|detach}_hook Philipp Rudo
2017-02-28 14:50 ` [PATCH 06/11] Remove unused deprecated_context_hook Philipp Rudo
2017-02-28 14:50 ` [PATCH 11/11] Remove deprecated_print_frame_info_listing_hook Philipp Rudo
2017-02-28 14:50 ` [PATCH 05/11] Remove unused deprecated_readline{_begin|_end}_hook Philipp Rudo
2017-02-28 14:50 ` [PATCH 01/11] Remove unused deprecated_{pre|post}_add_symbol_hook Philipp Rudo
2017-02-28 14:50 ` [PATCH 09/11] Remove unused deprecated_call_command_hook Philipp Rudo
2017-02-28 14:50 ` [PATCH 02/11] Remove unused deprecated_ui_loop_hook Philipp Rudo
2017-02-28 15:28 ` [PATCH 00/11] Cleanup deprecated_*_hooks in defs.h Tom Tromey
2017-02-28 15:52 ` Simon Marchi
2017-03-01 9:11 ` Philipp Rudo
2017-03-01 22:17 ` Yao Qi
2017-02-28 18:01 ` [PATCH 04/11] Remove unused deprecated_interactive_hook Philipp Rudo
2017-02-28 19:37 ` [PATCH 00/11] Cleanup deprecated_*_hooks in defs.h Keith Seitz
2017-02-28 19:45 ` [PATCH 07/11] Remove unused deprecated_target_wait_hook Philipp Rudo
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=20170228145016.22604-1-prudo@linux.vnet.ibm.com \
--to=prudo@linux.vnet.ibm.com \
--cc=arnez@linux.vnet.ibm.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