Skip to content

Commit

Permalink
Also refresh TTY timestamps before "commandline -f repaint"
Browse files Browse the repository at this point in the history
As mentioned in 8a7c3ce (Don't abandon line after writing control sequences,
2024-04-06) we need to freshed stdout timestamps after writing to stdout
but before we might redraw, in particular when writing control sequences.

Commit a583fe7 ("commandline -f foo" to skip queue and execute immediately,
2024-04-08) made "commandline -f repaint" redraw immediately, while still
executing the bound shell command; at that time we have written "disabling"
sequences but not refreshed timestamps yet, so do that.

This is probably not needed for commands outside the repaint family.
Needless to say that this is messy, maybe we can simplify things in future.

Ref #10409 (comment)
  • Loading branch information
krobelus committed Apr 9, 2024
1 parent 64bc989 commit 1da2087
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions src/reader.rs
Original file line number Diff line number Diff line change
Expand Up @@ -909,6 +909,7 @@ pub fn reader_execute_readline_cmd(ch: CharEvent) {
if data.rls.is_none() {
data.rls = Some(ReadlineLoopState::new());
}
data.save_screen_state();
data.apply_commandline_state_changes();
data.handle_char_event(Some(ch));
data.update_commandline_state();
Expand Down

0 comments on commit 1da2087

Please sign in to comment.