Re: [PATCH] Add a timeout parameter to gdb_do_one_event

[Simon Marchi via Gdb-patches <gdb-patches@sourceware.org>]

On 2022-03-17 09:08, Patrick Monnerat via Gdb-patches wrote:
> Since commit b2d8657, having a per-interpreter event/command loop is not
> possible anymore.
> 
> As Insight uses a GUI that has its own event loop, gdb and GUI event
> loops have then to be "merged" (i.e.: work together). But this is
> problematic as gdb_do_one_event is not aware of this alternate event
> loop and thus may wait forever.
> 
> The solution is to implement a wait timeout to gdb_do_one_event. This
> cannot be done externally as gdb timers are event sources themselves.
> 
> The new parameter defaults to "no timeout": as it is used by Insight
> only, there is no need to update calls from the gdb source tree.

Sorry for the delay.  The patch is fine with me.  Pedro and Tom also
took a look at previous iterations and I didn't see any disagreement
either.  I don't remember, do you have push access?  Otherwise I can
push it for you.

Simon