diff options
author | noname@inventati.org <noname@inventati.org> | 2015-04-11 19:29:52 +0200 |
---|---|---|
committer | Roberto E. Vargas Caballero <k0ga@shike2.com> | 2015-04-13 09:21:03 +0200 |
commit | 6352502d644d8295ceb2cdf68a5ecbac0891d4a6 (patch) | |
tree | 5a7069813cb3650c87c95c97cca24fd5eec07260 /st.c | |
parent | b9390a54968c3bc4f4270afdcf5b85911df01611 (diff) | |
download | st-6352502d644d8295ceb2cdf68a5ecbac0891d4a6.tar.gz st-6352502d644d8295ceb2cdf68a5ecbac0891d4a6.tar.bz2 |
tresize: move for loop outside if
There is no need to check that slide > 0 before executing loop.
If slide <= 0, loop stops immediately.
Diffstat (limited to 'st.c')
-rw-r--r-- | st.c | 9 |
1 files changed, 4 insertions, 5 deletions
@@ -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)); } |