diff options
author | zeertzjq <zeertzjq@outlook.com> | 2023-02-04 20:14:31 +0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-02-04 20:14:31 +0800 |
commit | 69bb145cea56067e6e82ed0a130a51c0d611e540 (patch) | |
tree | cbb045aac106e922647bd2a24ea4dd7a4b1763d4 /src/nvim/ui_client.c | |
parent | 90333b24c3582cb017d823583d4896c8bbb8edb8 (diff) | |
download | rneovim-69bb145cea56067e6e82ed0a130a51c0d611e540.tar.gz rneovim-69bb145cea56067e6e82ed0a130a51c0d611e540.tar.bz2 rneovim-69bb145cea56067e6e82ed0a130a51c0d611e540.zip |
refactor(exit): pass error message to preserve_exit() (#22097)
Problem:
1. Some calls to preserve_exit() don't put a message in IObuff, so the
IObuff printed by preserve_exit() contains unrelated information.
2. If a TUI client runs out of memory or receives a deadly signal, the
error message is shown on alternate screen and cannot be easily seen
because the TUI exits alternate screen soon afterwards.
Solution:
Pass error message to preserve_exit() and exit alternate screen before
printing it.
Note that this doesn't fix the problem that server error messages cannot
be easily seen on exit. This is tracked in #21608 and #21843.
Diffstat (limited to 'src/nvim/ui_client.c')
-rw-r--r-- | src/nvim/ui_client.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/nvim/ui_client.c b/src/nvim/ui_client.c index b5c8dff412..58edd7aff3 100644 --- a/src/nvim/ui_client.c +++ b/src/nvim/ui_client.c @@ -122,7 +122,9 @@ void ui_client_run(bool remote_ui) void ui_client_stop(void) { - tui_stop(tui); + if (!tui_is_stopped(tui)) { + tui_stop(tui); + } } void ui_client_set_size(int width, int height) |