From 6352502d644d8295ceb2cdf68a5ecbac0891d4a6 Mon Sep 17 00:00:00 2001 From: "noname@inventati.org" Date: Sat, 11 Apr 2015 19:29:52 +0200 Subject: [PATCH] tresize: move for loop outside if There is no need to check that slide > 0 before executing loop. If slide <= 0, loop stops immediately. --- st.c | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) 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)); } -- 2.20.1