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 | |
| parent | b9390a54968c3bc4f4270afdcf5b85911df01611 (diff) | |
| download | st-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.c | 9 |
1 files changed, 4 insertions, 5 deletions
| @@ -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 | } |
