aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPedro Alves <palves@redhat.com>2016-03-09 18:24:59 +0000
committerPedro Alves <palves@redhat.com>2016-03-09 18:24:59 +0000
commit720d2e96b424bb4bb1f7021f86763ff3df078e2b (patch)
treeee12a5a6b66782b5dbdd43ebf9f8ba1140a4a3b5
parent608ff013aae02e3ac4687691db57bbcb8984fe18 (diff)
downloadgdb-720d2e96b424bb4bb1f7021f86763ff3df078e2b.zip
gdb-720d2e96b424bb4bb1f7021f86763ff3df078e2b.tar.gz
gdb-720d2e96b424bb4bb1f7021f86763ff3df078e2b.tar.bz2
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 <palves@redhat.com> * utils.c (prompt_for_continue): Update comments.
-rw-r--r--gdb/ChangeLog4
-rw-r--r--gdb/utils.c22
2 files changed, 12 insertions, 14 deletions
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 <palves@redhat.com>
+ * utils.c (prompt_for_continue): Update comments.
+
+2016-03-09 Pedro Alves <palves@redhat.com>
+
* 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. */