From: Pedro Alves Date: Wed, 9 Mar 2016 18:24:59 +0000 (+0000) Subject: Update prompt_for_continue comments X-Git-Tag: binutils-2_27~1104 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=720d2e96b424bb4bb1f7021f86763ff3df078e2b;p=platform%2Fupstream%2Fbinutils.git 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. --- diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 264c8a7..e715c97 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,5 +1,9 @@ 2016-03-09 Pedro Alves + * utils.c (prompt_for_continue): Update comments. + +2016-03-09 Pedro Alves + * event-top.c (async_annotation_suffix): Delete. (top_level_prompt, command_line_handler): Don't use 'async_annotation_suffix' and simplify. diff --git a/gdb/utils.c b/gdb/utils.c index 977314b..97e5133 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. */