commit 6352502d644d8295ceb2cdf68a5ecbac0891d4a6
Author: [email protected] <[email protected]>
Date:   Sat Apr 11 19:29:52 2015 +0200

    tresize: move for loop outside if
    
    There is no need to check that slide > 0 before executing loop.
    If slide <= 0, loop stops immediately.

diff --git a/st.c b/st.c
index 4ed8319..fb37eb5 100644
--- a/st.c
+++ b/st.c
@@ -2780,17 +2780,16 @@ tresize(int col, int row) {
        }
 
        /* free unneeded rows */
-       i = 0;
+       for(i = 0; i < slide; i++) {
+               free(term.line[i]);
+               free(term.alt[i]);
+       }
        if(slide > 0) {
                /*
                 * slide screen to keep cursor where we expect it -
                 * tscrollup would work here, but we can optimize to
                 * memmove because we're freeing the earlier lines
                 */
-               for(/* i = 0 */; i < slide; i++) {
-                       free(term.line[i]);
-                       free(term.alt[i]);
-               }
                memmove(term.line, term.line + slide, row * sizeof(Line));
                memmove(term.alt, term.alt + slide, row * sizeof(Line));
        }

Reply via email to