diff options
author | N-R-K <nrk@disroot.org> | 2022-04-27 01:25:11 +0000 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-04-27 01:25:11 +0000 |
commit | 29c6b1456e8bb46fbee0e8ea962553cb9f1230a3 (patch) | |
tree | 99f72eff053384c0251cac83722ceb5653ed80bf /image.c | |
parent | 7fb8a61c47b8968f3da7decaefbe55b621e68420 (diff) |
code-style: reduce some unnecessary if-elses (#261)
also change the condition inside img_frame_animate() to check for
positive value rather than comparing against 0.
Diffstat (limited to 'image.c')
-rw-r--r-- | image.c | 16 |
1 files changed, 4 insertions, 12 deletions
@@ -898,23 +898,15 @@ bool img_frame_navigate(img_t *img, int d) return false; d += img->multi.sel; - if (d < 0) - d = 0; - else if (d >= img->multi.cnt) - d = img->multi.cnt - 1; + d = MAX(0, MIN(d, img->multi.cnt - 1)); return img_frame_goto(img, d); } bool img_frame_animate(img_t *img) { - if (img->multi.cnt == 0) - return false; - - if (img->multi.sel + 1 >= img->multi.cnt) - img_frame_goto(img, 0); + if (img->multi.cnt > 0) + return img_frame_goto(img, (img->multi.sel + 1) % img->multi.cnt); else - img_frame_goto(img, img->multi.sel + 1); - img->dirty = true; - return true; + return false; } |