diff options
author | June McEnroe <june@causal.agency> | 2020-02-17 11:49:56 -0500 |
---|---|---|
committer | June McEnroe <june@causal.agency> | 2020-02-17 11:49:56 -0500 |
commit | fa29c3791119d9fafac1509d43e8da22c31cda6d (patch) | |
tree | 0da2ff38171ce93a286b20db7df5b36f741d9738 | |
parent | Various small cleanups (diff) | |
download | catgirl-fa29c3791119d9fafac1509d43e8da22c31cda6d.tar.gz catgirl-fa29c3791119d9fafac1509d43e8da22c31cda6d.zip |
Scroll the window only once during reflow
ncurses implements scrolling as a memmove of the array of lines pointers, which happens each time a line is added to the bottom of the window, causing a scroll. This would get noticeably slow if WindowLines were increased to just 1024. Should've used a ring buffer, I think.
-rw-r--r-- | ui.c | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/ui.c b/ui.c index bcb1003..938eda6 100644 --- a/ui.c +++ b/ui.c @@ -621,18 +621,21 @@ void uiFormat( static void reflow(struct Window *window) { werase(window->pad); - wmove(window->pad, WindowLines - 1, 0); + wmove(window->pad, 0, 0); + int flowed = 0; window->unreadLines = 0; for (size_t i = 0; i < BufferCap; ++i) { const char *line = bufferLine(&window->buffer, i); if (!line) continue; waddch(window->pad, '\n'); + int lines = 1 + wordWrap(window->pad, line); if (i >= (size_t)(BufferCap - window->unreadTotal)) { - window->unreadLines += 1 + wordWrap(window->pad, line); - } else { - wordWrap(window->pad, line); + window->unreadLines += lines; } + flowed += lines; } + wscrl(window->pad, -(WindowLines - 1 - flowed)); + wmove(window->pad, WindowLines - 1, RIGHT); } static void resize(void) { |