aboutsummaryrefslogtreecommitdiff
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
st-6352502d644d8295ceb2cdf68a5ecbac0891d4a6.zip
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) {
2780 } 2780 }
2781 2781
2782 /* free unneeded rows */ 2782 /* free unneeded rows */
2783 i = 0; 2783 for(i = 0; i < slide; i++) {
2784 free(term.line[i]);
2785 free(term.alt[i]);
2786 }
2784 if(slide > 0) { 2787 if(slide > 0) {
2785 /* 2788 /*
2786 * slide screen to keep cursor where we expect it - 2789 * slide screen to keep cursor where we expect it -
2787 * tscrollup would work here, but we can optimize to 2790 * tscrollup would work here, but we can optimize to
2788 * memmove because we're freeing the earlier lines 2791 * memmove because we're freeing the earlier lines
2789 */ 2792 */
2790 for(/* i = 0 */; i < slide; i++) {
2791 free(term.line[i]);
2792 free(term.alt[i]);
2793 }
2794 memmove(term.line, term.line + slide, row * sizeof(Line)); 2793 memmove(term.line, term.line + slide, row * sizeof(Line));
2795 memmove(term.alt, term.alt + slide, row * sizeof(Line)); 2794 memmove(term.alt, term.alt + slide, row * sizeof(Line));
2796 } 2795 }