Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Commit 21267b6

Browse files
committed
Fix \watch's interaction with libedit on ^C.
When you hit ^C, the terminal driver in Unix-like systems echoes "^C" as well as sending an interrupt signal (depending on stty settings). At least libedit (but maybe also libreadline) is then confused about the current cursor location, and corrupts the display if you try to scroll back. Fix, by moving to a new line before the next prompt is displayed. Back-patch to all supported released. Author: Pavel Stehule <pavel.stehule@gmail.com> Reported-by: Tom Lane <tgl@sss.pgh.pa.us> Discussion: https://postgr.es/m/3278793.1626198638%40sss.pgh.pa.us
1 parent 301b06d commit 21267b6

File tree

1 file changed

+12
-0
lines changed

1 file changed

+12
-0
lines changed

src/bin/psql/command.c

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5145,6 +5145,18 @@ do_watch(PQExpBuffer query_buf, double sleep)
51455145
pclose(pagerpipe);
51465146
restore_sigpipe_trap();
51475147
}
5148+
else
5149+
{
5150+
/*
5151+
* If the terminal driver echoed "^C", libedit/libreadline might be
5152+
* confused about the cursor position. Therefore, inject a newline
5153+
* before the next prompt is displayed. We only do this when not
5154+
* using a pager, because pagers are expected to restore the screen to
5155+
* a sane state on exit.
5156+
*/
5157+
fprintf(stdout, "\n");
5158+
fflush(stdout);
5159+
}
51485160

51495161
#ifdef HAVE_POSIX_DECL_SIGWAIT
51505162
/* Disable the interval timer. */

0 commit comments

Comments
 (0)