dany/libwebp
1
0
Fork 0
mirror of https://github.com/webmproject/libwebp.git synced 2025-04-03 07:35:59 +02:00
Code Issues Actions Packages Projects Releases Wiki Activity

FlattenSimilarBlocks should only be tried when blending is possible.

Browse Source 
This is because, FlattenSimilarBlocks() replaces some opaque pixels by
transparent ones. This results in an equivalent output only if blending
is turned on for the current frame.

(cherry picked from commit 5cccdadf2e75e4eae684951fb507b2bebbda66cb)

Change-Id: I05612c952fdbd4b3a6e0ac9f3a7d49822f0cfb9b
This commit is contained in:
Urvang Joshi 2014-11-05 10:44:07 -08:00 committed by James Zern
parent f55ebbba82
commit 84ecd9d85c
1 changed files with 4 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
examples/gif2webp_util.c
View File

@ -671,11 +671,9 @@ static WebPEncodingError GenerateCandidates(
}
}
if (candidate_lossy->evaluate) {
if (!is_key_frame) {
// For lossy compression of a frame, it's better to:
// * Replace transparent pixels of 'curr' with actual RGB values,
// whenever possible, and
// * Replace similar blocks of pixels by a transparent block.
if (use_blending) {
// For lossy compression of a frame, it's better to replace similar blocks
// of pixels by a transparent block.
if (!curr_canvas_saved) { // save if not already done so.
CopyPixels(curr_canvas, curr_canvas_tmp);
}
@ -684,7 +682,7 @@ static WebPEncodingError GenerateCandidates(
error_code = EncodeCandidate(sub_frame, rect, config_lossy, use_blending,
duration, candidate_lossy);
if (error_code != VP8_ENC_OK) return error_code;
if (!is_key_frame) {
if (use_blending) {
CopyPixels(curr_canvas_tmp, curr_canvas); // restore
}
}