From 720d2e96b424bb4bb1f7021f86763ff3df078e2b Mon Sep 17 00:00:00 2001 From: Pedro Alves Date: Wed, 9 Mar 2016 18:24:59 +0000 Subject: [PATCH] Update prompt_for_continue comments These comments are out of date -- we no longer call gdb_readline. And I think that mentioning the event loop is more useful here than whatever GO32 issue had with gdb_readline, which may even no longer be an issue. gdb/ChangeLog: 2016-03-09 Pedro Alves * utils.c (prompt_for_continue): Update comments. --- gdb/ChangeLog | 4 ++++ gdb/utils.c | 22 ++++++++-------------- 2 files changed, 12 insertions(+), 14 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 264c8a74a44..e715c9712f9 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,7 @@ +2016-03-09 Pedro Alves + + * utils.c (prompt_for_continue): Update comments. + 2016-03-09 Pedro Alves * event-top.c (async_annotation_suffix): Delete. diff --git a/gdb/utils.c b/gdb/utils.c index 977314b95a2..97e5133f7a4 100644 --- a/gdb/utils.c +++ b/gdb/utils.c @@ -1808,7 +1808,9 @@ set_screen_width_and_height (int width, int height) } /* Wait, so the user can read what's on the screen. Prompt the user - to continue by pressing RETURN. */ + to continue by pressing RETURN. 'q' is also provided because + telling users what to do in the prompt is more user-friendly than + expecting them to think of Ctrl-C/SIGINT. */ static void prompt_for_continue (void) @@ -1829,9 +1831,9 @@ prompt_for_continue (void) if (annotation_level > 1) strcat (cont_prompt, "\n\032\032prompt-for-continue\n"); - /* We must do this *before* we call gdb_readline, else it will eventually - call us -- thinking that we're trying to print beyond the end of the - screen. */ + /* We must do this *before* we call gdb_readline_wrapper, else it + will eventually call us -- thinking that we're trying to print + beyond the end of the screen. */ reinitialize_more_filter (); immediate_quit++; @@ -1840,16 +1842,8 @@ prompt_for_continue (void) /* We'll need to handle input. */ target_terminal_ours (); - /* On a real operating system, the user can quit with SIGINT. - But not on GO32. - - 'q' is provided on all systems so users don't have to change habits - from system to system, and because telling them what to do in - the prompt is more user-friendly than expecting them to think of - SIGINT. */ - /* Call readline, not gdb_readline, because GO32 readline handles control-C - whereas control-C to gdb_readline will cause the user to get dumped - out to DOS. */ + /* Call gdb_readline_wrapper, not readline, in order to keep an + event loop running. */ ignore = gdb_readline_wrapper (cont_prompt); /* Add time spend in this routine to prompt_for_continue_wait_time. */ -- 2.30.2