Code

list_window: page up goes too far
authorJeffrey Middleton <jefromi@gmail.com>
Fri, 27 Feb 2009 08:11:24 +0000 (09:11 +0100)
committerMax Kellermann <max@duempel.org>
Fri, 27 Feb 2009 08:11:24 +0000 (09:11 +0100)
There's a sign error in the page up (previous screen) - instead of
going up by one less than the number of rows, it goes up by one
more. Patch attached, though it's just a single character change. I'm
kind of weirded out that I never noticed this before, since it seems
to have been like this forever!

src/list_window.c

index c09844082ac2e20f5ec3194d5bd9b5d5cb6d5378..17107d67768bbc44c15b50d4d01bf9e4815b6bfd 100644 (file)
@@ -186,7 +186,7 @@ list_window_previous_page(struct list_window *lw)
        if (lw->rows < 2)
                return;
        if (lw->selected > lw->rows - 1)
-               list_window_set_selected(lw, lw->selected - lw->rows - 1);
+               list_window_set_selected(lw, lw->selected - lw->rows + 1);
        else
                list_window_first(lw);
 }