diff --git a/gdb/ChangeLog b/gdb/ChangeLog index aa23a65faf7..9d3ff4acd66 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,8 @@ +2016-04-12 Pedro Alves + + * event-top.c (stdin_event_handler): Call QUIT; + (prompt_for_continue): Don't run with immediate_quit set. + 2016-04-12 Pedro Alves * tui/tui-io.c (tui_redisplay_readline): Check diff --git a/gdb/event-top.c b/gdb/event-top.c index eb4f0b9fc1d..a119fe98734 100644 --- a/gdb/event-top.c +++ b/gdb/event-top.c @@ -403,6 +403,15 @@ stdin_event_handler (int error, gdb_client_data client_data) } else { + /* This makes sure a ^C immediately followed by further input is + always processed in that order. E.g,. with input like + "^Cprint 1\n", the SIGINT handler runs, marks the async signal + handler, and then select/poll may return with stdin ready, + instead of -1/EINTR. The + gdb.base/double-prompt-target-event-error.exp test exercises + this. */ + QUIT; + do { call_stdin_event_handler_again_p = 0; diff --git a/gdb/utils.c b/gdb/utils.c index 62e169ad8f5..3c7a29cbb8b 100644 --- a/gdb/utils.c +++ b/gdb/utils.c @@ -1865,9 +1865,6 @@ prompt_for_continue (void) beyond the end of the screen. */ reinitialize_more_filter (); - immediate_quit++; - QUIT; - /* We'll need to handle input. */ target_terminal_ours (); @@ -1895,7 +1892,6 @@ prompt_for_continue (void) throw_quit ("Quit"); xfree (ignore); } - immediate_quit--; /* Now we have to do this again, so that GDB will know that it doesn't need to save the ---Type --- line at the top of the screen. */