diff options
author | Tom Tromey <tromey@adacore.com> | 2024-09-25 08:27:51 -0600 |
---|---|---|
committer | Tom Tromey <tromey@adacore.com> | 2024-11-01 11:06:47 -0600 |
commit | 76367d23146136b457b1494285b807437a9fec60 (patch) | |
tree | 29cec25266e8d522ba7a135659699542885af03d | |
parent | 893e4fd6231922495175b701a6c44b665e91cd7e (diff) | |
download | gdb-76367d23146136b457b1494285b807437a9fec60.zip gdb-76367d23146136b457b1494285b807437a9fec60.tar.gz gdb-76367d23146136b457b1494285b807437a9fec60.tar.bz2 |
Add gdb.events.tui_enabled
This adds a new event source so that Python scripts can track whether
or not the TUI is presently enabled.
v2 of the patch renames "status" -> "enabled".
Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=32162
Reviewed-By: Eli Zaretskii <eliz@gnu.org>
Reviewed-by: Keith Seitz <keiths@redhat.com>
-rw-r--r-- | gdb/NEWS | 2 | ||||
-rw-r--r-- | gdb/doc/python.texi | 10 | ||||
-rw-r--r-- | gdb/observable.c | 1 | ||||
-rw-r--r-- | gdb/observable.h | 2 | ||||
-rw-r--r-- | gdb/python/py-all-events.def | 1 | ||||
-rw-r--r-- | gdb/python/py-event-types.def | 5 | ||||
-rw-r--r-- | gdb/python/py-tui.c | 28 | ||||
-rw-r--r-- | gdb/testsuite/gdb.python/tui-window.exp | 8 | ||||
-rw-r--r-- | gdb/testsuite/gdb.python/tui-window.py | 11 | ||||
-rw-r--r-- | gdb/tui/tui.c | 5 |
10 files changed, 73 insertions, 0 deletions
@@ -56,6 +56,8 @@ ** Added gdb.record.clear. Clears the trace data of the current recording. This forces re-decoding of the trace for successive commands. + ** Added the new event source gdb.tui_enabled. + * Debugger Adapter Protocol changes ** The "scopes" request will now return a scope holding global diff --git a/gdb/doc/python.texi b/gdb/doc/python.texi index 22f0e6c..b68fe04 100644 --- a/gdb/doc/python.texi +++ b/gdb/doc/python.texi @@ -4081,6 +4081,16 @@ The @code{gdb.Progspace} that is about to be removed from @value{GDBN}. @end defvar +@item events.tui_enabled +This is emitted when the TUI is enabled or disabled. The event is of +type @code{gdb.TuiEnabledEvent}, which has a single read-only +attribute: + +@defvar TuiStatusEvent.enabled +If the TUI has just been enabled, this is @code{True}; otherwise it is +@code{False}. +@end defvar + @end table @node Threads In Python diff --git a/gdb/observable.c b/gdb/observable.c index 7816d21..cc1b01d 100644 --- a/gdb/observable.c +++ b/gdb/observable.c @@ -75,6 +75,7 @@ DEFINE_OBSERVABLE (target_pre_wait); DEFINE_OBSERVABLE (target_post_wait); DEFINE_OBSERVABLE (new_program_space); DEFINE_OBSERVABLE (free_program_space); +DEFINE_OBSERVABLE (tui_enabled); } /* namespace observers */ } /* namespace gdb */ diff --git a/gdb/observable.h b/gdb/observable.h index 2aa3ef3..06c0cf1 100644 --- a/gdb/observable.h +++ b/gdb/observable.h @@ -253,6 +253,8 @@ extern observable <program_space */* pspace */> new_program_space; /* The program space PSPACE is about to be deleted. */ extern observable <program_space */* pspace */> free_program_space; +extern observable<bool /* enabled */> tui_enabled; + } /* namespace observers */ } /* namespace gdb */ diff --git a/gdb/python/py-all-events.def b/gdb/python/py-all-events.def index 4df5541..f315e75 100644 --- a/gdb/python/py-all-events.def +++ b/gdb/python/py-all-events.def @@ -45,3 +45,4 @@ GDB_PY_DEFINE_EVENT(connection_removed) GDB_PY_DEFINE_EVENT(executable_changed) GDB_PY_DEFINE_EVENT(new_progspace) GDB_PY_DEFINE_EVENT(free_progspace) +GDB_PY_DEFINE_EVENT(tui_enabled) diff --git a/gdb/python/py-event-types.def b/gdb/python/py-event-types.def index f43a51b..e22f042 100644 --- a/gdb/python/py-event-types.def +++ b/gdb/python/py-event-types.def @@ -140,3 +140,8 @@ GDB_PY_DEFINE_EVENT_TYPE (free_progspace, "FreeProgspaceEvent", "GDB free Progspace event object", event_object_type); + +GDB_PY_DEFINE_EVENT_TYPE (tui_enabled, + "TuiEnabledEvent", + "GDB TUI enabled event object", + event_object_type); diff --git a/gdb/python/py-tui.c b/gdb/python/py-tui.c index afa6f36..09f0a60 100644 --- a/gdb/python/py-tui.c +++ b/gdb/python/py-tui.c @@ -36,6 +36,9 @@ #include "tui/tui-layout.h" #include "tui/tui-wingeneral.h" #include "tui/tui-winsource.h" +#include "observable.h" +#include "py-events.h" +#include "py-event.h" class tui_py_window; @@ -616,6 +619,29 @@ PyTypeObject gdbpy_tui_window_object_type = 0, /* tp_alloc */ }; +/* Called when TUI is enabled or disabled. */ + +static void +gdbpy_tui_enabled (bool state) +{ + gdbpy_enter enter_py; + + if (evregpy_no_listeners_p (gdb_py_events.tui_enabled)) + return; + + gdbpy_ref<> event_obj = create_event_object (&tui_enabled_event_object_type); + if (event_obj == nullptr) + { + gdbpy_print_stack (); + return; + } + + gdbpy_ref<> code (PyBool_FromLong (state)); + if (evpy_add_attribute (event_obj.get (), "enabled", code.get ()) < 0 + || evpy_emit_event (event_obj.get (), gdb_py_events.tui_enabled) < 0) + gdbpy_print_stack (); +} + #endif /* TUI */ /* Initialize this module. */ @@ -627,6 +653,8 @@ gdbpy_initialize_tui () gdbpy_tui_window_object_type.tp_new = PyType_GenericNew; if (gdbpy_type_ready (&gdbpy_tui_window_object_type) < 0) return -1; + + gdb::observers::tui_enabled.attach (gdbpy_tui_enabled, "py-tui"); #endif /* TUI */ return 0; diff --git a/gdb/testsuite/gdb.python/tui-window.exp b/gdb/testsuite/gdb.python/tui-window.exp index e7ff162..c521dcc 100644 --- a/gdb/testsuite/gdb.python/tui-window.exp +++ b/gdb/testsuite/gdb.python/tui-window.exp @@ -43,6 +43,9 @@ if {![Term::enter_tui]} { return } +Term::command "python print('tui_enabled=' + str(tui_enabled))" +Term::check_contents "tui start event" "tui_enabled=True" + Term::command "layout test" Term::check_contents "test title" \ "This Is The Title" @@ -63,3 +66,8 @@ Term::resize 51 51 Term::check_contents "Window was updated" "Test: 2" Term::command "layout fail" + +Term::command "tui disable" +gdb_test "python print('tui_enabled=' + str(tui_enabled))" \ + "tui_enabled=False" \ + "tui disable event" diff --git a/gdb/testsuite/gdb.python/tui-window.py b/gdb/testsuite/gdb.python/tui-window.py index d031f72..0661acc 100644 --- a/gdb/testsuite/gdb.python/tui-window.py +++ b/gdb/testsuite/gdb.python/tui-window.py @@ -20,6 +20,9 @@ import gdb the_window = None +tui_enabled = False + + class TestWindow: def __init__(self, win): global the_window @@ -62,3 +65,11 @@ def change_window_title(): gdb.register_window_type("fail", failwin) + + +def set_tui_enabled(ev): + global tui_enabled + tui_enabled = ev.enabled + + +gdb.events.tui_enabled.connect(set_tui_enabled) diff --git a/gdb/tui/tui.c b/gdb/tui/tui.c index bc96cd8..59aa1bc 100644 --- a/gdb/tui/tui.c +++ b/gdb/tui/tui.c @@ -40,6 +40,7 @@ #include "terminal.h" #include "top.h" #include "ui.h" +#include "observable.h" #include <fcntl.h> @@ -509,6 +510,8 @@ tui_enable (void) /* Update gdb's knowledge of its terminal. */ gdb_save_tty_state (); tui_update_gdb_sizes (); + + gdb::observers::tui_enabled.notify (true); } /* Leave the tui mode. @@ -547,6 +550,8 @@ tui_disable (void) tui_active = false; tui_update_gdb_sizes (); + + gdb::observers::tui_enabled.notify (false); } /* Command wrapper for enabling tui mode. */ |