aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authornoname@inventati.org <noname@inventati.org>2015-04-11 19:29:52 +0200
committerRoberto E. Vargas Caballero <k0ga@shike2.com>2015-04-13 09:21:03 +0200
commit6352502d644d8295ceb2cdf68a5ecbac0891d4a6 (patch)
tree5a7069813cb3650c87c95c97cca24fd5eec07260
parentb9390a54968c3bc4f4270afdcf5b85911df01611 (diff)
downloadst-6352502d644d8295ceb2cdf68a5ecbac0891d4a6.tar.gz
tresize: move for loop outside if
There is no need to check that slide > 0 before executing loop. If slide <= 0, loop stops immediately.
-rw-r--r--st.c9
1 files 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));
}
remember that computers suck.