uefi-ref/binutils-gdb
1
0
Fork 0
mirror of https://sourceware.org/git/binutils-gdb.git synced 2026-08-27 00:26:02 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

gdb: add interp::on_tsv_deleted method

Browse source 
Same idea as previous patches, but for tsv_deleted.

Change-Id: I71b0502b493da7b6e293bee02aeca98de83d4b75
This commit is contained in:
Simon Marchi 2023-04-21 09:45:30 -04:00
parent bf506f275a
commit f0dffaff4f
7 changed files with 29 additions and 29 deletions
Download patch file Download diff file Expand all files Collapse all files

8
gdb/interps.c
View file

@ -543,6 +543,14 @@ interps_notify_tsv_created (const trace_state_variable *tsv)
interps_notify (&interp::on_tsv_created, tsv);
}
/* See interps.h. */
void
interps_notify_tsv_deleted (const trace_state_variable *tsv)
{
interps_notify (&interp::on_tsv_deleted, tsv);
}
/* This just adds the "interpreter-exec" command. */
void _initialize_interpreter ();
void

8
gdb/interps.h
View file

@ -160,6 +160,9 @@ public:
/* Notify the interpreter that trace state variable TSV was created. */
virtual void on_tsv_created (const trace_state_variable *tsv) {}
/* Notify the interpreter that trace state variable TSV was deleted. */
virtual void on_tsv_deleted (const trace_state_variable *tsv) {}
private:
/* The memory for this is static, it comes from literal strings (e.g. "cli"). */
const char *m_name;
@ -321,6 +324,11 @@ extern void interps_notify_traceframe_changed (int tfnum, int tpnum);
/* Notify all interpreters that trace state variable TSV was created. */
extern void interps_notify_tsv_created (const trace_state_variable *tsv);
/* Notify all interpreters that trace state variable TSV was deleted.
If TSV is nullptr, it means that all trace state variables were deleted. */
extern void interps_notify_tsv_deleted (const trace_state_variable *tsv);
/* well-known interpreters */
#define INTERP_CONSOLE "console"
#define INTERP_MI2 "mi2"

32
gdb/mi/mi-interp.c
View file

@ -60,7 +60,6 @@ static int mi_interp_query_hook (const char *ctlstr, va_list ap)
static void mi_insert_notify_hooks (void);
static void mi_remove_notify_hooks (void);
static void mi_tsv_deleted (const struct trace_state_variable *tsv);
static void mi_tsv_modified (const struct trace_state_variable *tsv);
static void mi_breakpoint_created (struct breakpoint *b);
static void mi_breakpoint_deleted (struct breakpoint *b);
@ -525,29 +524,19 @@ mi_interp::on_tsv_created (const trace_state_variable *tsv)
gdb_flush (this->event_channel);
}
/* Emit notification on deleting a trace state variable. */
static void
mi_tsv_deleted (const struct trace_state_variable *tsv)
void
mi_interp::on_tsv_deleted (const trace_state_variable *tsv)
{
SWITCH_THRU_ALL_UIS ()
{
struct mi_interp *mi = as_mi_interp (top_level_interpreter ());
target_terminal::scoped_restore_terminal_state term_state;
target_terminal::ours_for_output ();
if (mi == NULL)
continue;
if (tsv != nullptr)
gdb_printf (this->event_channel, "tsv-deleted,name=\"%s\"",
tsv->name.c_str ());
else
gdb_printf (this->event_channel, "tsv-deleted");
target_terminal::scoped_restore_terminal_state term_state;
target_terminal::ours_for_output ();
if (tsv != NULL)
gdb_printf (mi->event_channel, "tsv-deleted,"
"name=\"%s\"", tsv->name.c_str ());
else
gdb_printf (mi->event_channel, "tsv-deleted");
gdb_flush (mi->event_channel);
}
gdb_flush (this->event_channel);
}
/* Emit notification on modifying a trace state variable. */
@ -1033,7 +1022,6 @@ _initialize_mi_interp ()
interp_factory_register (INTERP_MI4, mi_interp_factory);
interp_factory_register (INTERP_MI, mi_interp_factory);
gdb::observers::tsv_deleted.attach (mi_tsv_deleted, "mi-interp");
gdb::observers::tsv_modified.attach (mi_tsv_modified, "mi-interp");
gdb::observers::breakpoint_created.attach (mi_breakpoint_created,
"mi-interp");

1
gdb/mi/mi-interp.h
View file

@ -64,6 +64,7 @@ public:
void on_about_to_proceed () override;
void on_traceframe_changed (int tfnum, int tpnum) override;
void on_tsv_created (const trace_state_variable *tsv) override;
void on_tsv_deleted (const trace_state_variable *tsv) override;
/* MI's output channels */
mi_console_file *out;

1
gdb/observable.c
View file

@ -62,7 +62,6 @@ DEFINE_OBSERVABLE (memory_changed);
DEFINE_OBSERVABLE (before_prompt);
DEFINE_OBSERVABLE (gdb_datadir_changed);
DEFINE_OBSERVABLE (command_param_changed);
DEFINE_OBSERVABLE (tsv_deleted);
DEFINE_OBSERVABLE (tsv_modified);
DEFINE_OBSERVABLE (inferior_call_pre);
DEFINE_OBSERVABLE (inferior_call_post);

4
gdb/observable.h
View file

@ -190,10 +190,6 @@ extern observable<> gdb_datadir_changed;
extern observable<const char */* param */, const char */* value */>
command_param_changed;
/* The trace state variable TSV is deleted. If TSV is NULL, all
trace state variables are deleted. */
extern observable<const struct trace_state_variable */* tsv */> tsv_deleted;
/* The trace state value TSV is modified. */
extern observable<const struct trace_state_variable */* tsv */> tsv_modified;

4
gdb/tracepoint.c
View file

@ -289,7 +289,7 @@ delete_trace_state_variable (const char *name)
for (auto it = tvariables.begin (); it != tvariables.end (); it++)
if (it->name == name)
{
gdb::observers::tsv_deleted.notify (&*it);
interps_notify_tsv_deleted (&*it);
tvariables.erase (it);
return;
}
@ -388,7 +388,7 @@ delete_trace_variable_command (const char *args, int from_tty)
if (query (_("Delete all trace state variables? ")))
tvariables.clear ();
dont_repeat ();
gdb::observers::tsv_deleted.notify (NULL);
interps_notify_tsv_deleted (nullptr);
return;
}