diff options
| author | Michael Drake <tlsa@netsurf-browser.org> | 2021-04-04 12:11:23 +0100 |
|---|---|---|
| committer | Michael Drake <tlsa@netsurf-browser.org> | 2021-04-06 09:03:19 +0100 |
| commit | 45c0d2812d13d110b151e4b60787275572c65e69 (patch) | |
| tree | b86754b0db2ad05d969e7590dcafe566defa985c /src | |
| parent | c2074de22a8d77ff8e688a3f22132f6d0ee44c20 (diff) | |
| download | libnsgif-45c0d2812d13d110b151e4b60787275572c65e69.tar.gz libnsgif-45c0d2812d13d110b151e4b60787275572c65e69.tar.bz2 | |
gif: Frame decoding: Rearrange for readability.
Diffstat (limited to 'src')
| -rw-r--r-- | src/libnsgif.c | 34 |
1 files changed, 16 insertions, 18 deletions
diff --git a/src/libnsgif.c b/src/libnsgif.c index 0874ec9..1540e1d 100644 --- a/src/libnsgif.c +++ b/src/libnsgif.c @@ -641,7 +641,7 @@ gif__decode(gif_animation *gif, unsigned int transparency_index; const uint8_t *stack_base; const uint8_t *stack_pos; - uint32_t written = 0; + uint32_t available = 0; gif_result ret = GIF_OK; lzw_result res; @@ -672,23 +672,9 @@ gif__decode(gif_animation *gif, x = width; while (x > 0) { - if (written > 0) { - unsigned burst_bytes = written; - if (burst_bytes > x) { - burst_bytes = x; - } - x -= burst_bytes; - written -= burst_bytes; - while (burst_bytes-- > 0) { - register unsigned int colour; - colour = *stack_pos++; - if (colour != transparency_index) { - *frame_scanline = colour_table[colour]; - } - frame_scanline++; - } - } else { - res = lzw_decode(gif->lzw_ctx, &written); + unsigned row_available; + if (available == 0) { + res = lzw_decode(gif->lzw_ctx, &available); if (res != LZW_OK) { /* Unexpected end of frame, try to recover */ if (res == LZW_OK_EOD) { @@ -700,6 +686,18 @@ gif__decode(gif_animation *gif, } stack_pos = stack_base; } + + row_available = x < available ? x : available; + x -= row_available; + available -= row_available; + while (row_available-- > 0) { + register unsigned int colour; + colour = *stack_pos++; + if (colour != transparency_index) { + *frame_scanline = colour_table[colour]; + } + frame_scanline++; + } } } return ret; |