Skip to content

Commit

Permalink
debugger: add a comment about why we don't support threaded mode in t…
Browse files Browse the repository at this point in the history
…he debugger

Signed-off-by: Balazs Scheidler <[email protected]>
  • Loading branch information
bazsi committed Oct 6, 2024
1 parent fd46324 commit cf46e3d
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion lib/debugger/debugger-main.c
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,14 @@ detach_debugger(gpointer user_data)
void
debugger_start(MainLoop *main_loop, GlobalConfig *cfg)
{
/* we don't support threaded mode (yet), force it to non-threaded */
/* we don't support threaded mode (yet) as breakpoints would need to be
* worker thread aware, but at the moment we only support a single
* pending breakpoint at a time, force it to non-threaded.
*
* The solution is to have the worker side state to be per-thread and
* handle multiple pending breakpoints.
*/

cfg->threaded = FALSE;
current_debugger = debugger_new(main_loop, cfg);
debugger_start_console(current_debugger);
Expand Down

0 comments on commit cf46e3d

Please sign in to comment.