I suggest we change that to simply:
/* If running interactively, warn the user a trace run is ongoing.
She may want to cancel detaching instead. */
if (current_trace_status ()->running && from_tty)
{
if (current_trace_status ()->disconnected_tracing)
{
if (!query (_("Trace is running and will continue after detach; detach anyway? ")))
error (_("Not confirmed."));
}
else
{
if (!query (_("Trace is running but will stop on detach; detach anyway? ")))
error (_("Not confirmed."));
}
}
- simpler, more coherent, less explaining, less confusing.