Re: [PATCH 1/7] Merge async and sync code paths some more

[Simon Marchi <simon.marchi@ericsson.com>]

On 15-08-12 01:01 PM, Pedro Alves wrote:
> diff --git a/gdb/mi/mi-interp.c b/gdb/mi/mi-interp.c
> index 99ce385..bdd8e21 100644
> --- a/gdb/mi/mi-interp.c
> +++ b/gdb/mi/mi-interp.c
> @@ -309,16 +309,8 @@ mi_execute_command_wrapper (const char *cmd)
>  static void
>  mi_on_sync_execution_done (void)
>  {
> -  /* MI generally prints a prompt after a command, indicating it's
> -     ready for further input.  However, due to an historical wart, if
> -     MI async, and a (CLI) synchronous command was issued, then we
> -     will print the prompt right after printing "^running", even if we
> -     cannot actually accept any input until the target stops.  See
> -     mi_on_resume.  However, if the target is async but MI is sync,
> -     then we need to output the MI prompt now, to replicate gdb's
> -     behavior when neither the target nor MI are async.  (Note this
> -     observer is only called by the asynchronous target event handling
> -     code.)  */
> +  /* If we MI is sync, then output the MI prompt now, indicating we're
> +     ready for further input.  */

"If we MI is sync"