From: noname@inventati.org <noname@inventati.org>
Date: Sat, 11 Apr 2015 17:29:52 +0000 (+0200)
Subject: tresize: move for loop outside if
X-Git-Url: https://git.xinqibao.xyz/st.git/commitdiff_plain/6352502d644d8295ceb2cdf68a5ecbac0891d4a6?ds=inline;hp=b9390a54968c3bc4f4270afdcf5b85911df01611

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));
 	}