optimized caret scrolling: this is a costly operation and is now done only once per keypress

* Esp. costly since Scintilla 5.
* We now avoid any Scintilla message that automatically scrolls the caret (makes the
  caret visible) and instead call SCI_SCROLLCARET only once after every keypress in the
  interface implementation.
* From nowon, use
  * SCI_SETEMPTYSELECTION instead of SCI_GOTOPOS
  * SCI_SETEMPTYSELECTION(SCI_POSITIONFROMLINE(...)) instead of SCI_GOTOLINE
  * SCI_SETSELECTIONSTART and SCI_SETSELECTIONEND instead of SCI_SETSEL
* With these optimizations we are significantly faster than before
  the Scintilla upgrade (6e67f5a682ff46d69888fec61b94bf45cec46721).
  It is now even safe to execute the Gtk test suite during CI.

1 files changed, 2 insertions, 2 deletions

diff --git a/src/help.c b/src/help.c
index fe6df1d..6f6f454 100644
--- a/src/help.c
+++ b/src/help.c
@@ -307,9 +307,9 @@ teco_state_help_done(teco_machine_main_t *ctx, const teco_string_t *str, GError
 	 * multiple topics in the same buffer without
 	 * closing it first.
 	 */
-	undo__teco_interface_ssm(SCI_GOTOPOS,
+	undo__teco_interface_ssm(SCI_SETEMPTYSELECTION,
 	                         teco_interface_ssm(SCI_GETCURRENTPOS, 0, 0), 0);
-	teco_interface_ssm(SCI_GOTOPOS, topic->pos, 0);
+	teco_interface_ssm(SCI_SETEMPTYSELECTION, topic->pos, 0);
 
 	return &teco_state_start;
 }