dany/libwebp
1
0
Fork 0
mirror of https://github.com/webmproject/libwebp.git synced 2024-12-28 14:38:21 +01:00
Code Issues Actions Packages Projects Releases Wiki Activity

Re-enable encoding of alpha plane with color cache for next release.

Browse Source 
This is a revert of: https://chromium-review.googlesource.com/#/c/73607/

Change-Id: I7ec45277d73608d77d5e873290c6c185caa30c32
This commit is contained in:
Urvang Joshi 2015-11-12 16:06:25 -08:00 committed by James Zern
parent 1f7148a4f1
commit 6ecd72f845
3 changed files with 8 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/enc/alpha.c
View File

@ -74,11 +74,7 @@ static int EncodeLossless(const uint8_t* const data, int width, int height,
config.quality = 8.f * effort_level; config.quality = 8.f * effort_level;
assert(config.quality >= 0 && config.quality <= 100.f); assert(config.quality >= 0 && config.quality <= 100.f);
// TODO(urvang): Temporary fix to avoid generating images that trigger ok = (VP8LEncodeStream(&config, &picture, bw) == VP8_ENC_OK);
// a decoder bug related to alpha with color cache.
// See: https://code.google.com/p/webp/issues/detail?id=239
// Need to re-enable this later.
ok = (VP8LEncodeStream(&config, &picture, bw, 0 /*use_cache*/) == VP8_ENC_OK);
WebPPictureFree(&picture); WebPPictureFree(&picture);
ok = ok && !bw->error_; ok = ok && !bw->error_;
if (!ok) { if (!ok) {

13
src/enc/vp8l.c
View File

@ -823,8 +823,7 @@ static WebPEncodingError EncodeImageInternal(VP8LBitWriter* const bw,
VP8LHashChain* const hash_chain, VP8LHashChain* const hash_chain,
VP8LBackwardRefs refs_array[2], VP8LBackwardRefs refs_array[2],
int width, int height, int quality, int width, int height, int quality,
int low_effort, int low_effort, int* cache_bits,
int use_cache, int* cache_bits,
int histogram_bits, int histogram_bits,
size_t init_byte_position, size_t init_byte_position,
int* const hdr_size, int* const hdr_size,
@ -856,7 +855,7 @@ static WebPEncodingError EncodeImageInternal(VP8LBitWriter* const bw,
goto Error; goto Error;
} }
*cache_bits = use_cache ? MAX_COLOR_CACHE_BITS : 0; *cache_bits = MAX_COLOR_CACHE_BITS;
// 'best_refs' is the reference to the best backward refs and points to one // 'best_refs' is the reference to the best backward refs and points to one
// of refs_array[0] or refs_array[1]. // of refs_array[0] or refs_array[1].
// Calculate backward references from ARGB image. // Calculate backward references from ARGB image.
@ -1375,7 +1374,7 @@ static void VP8LEncoderDelete(VP8LEncoder* enc) {
WebPEncodingError VP8LEncodeStream(const WebPConfig* const config, WebPEncodingError VP8LEncodeStream(const WebPConfig* const config,
const WebPPicture* const picture, const WebPPicture* const picture,
VP8LBitWriter* const bw, int use_cache) { VP8LBitWriter* const bw) {
WebPEncodingError err = VP8_ENC_OK; WebPEncodingError err = VP8_ENC_OK;
const int quality = (int)config->quality; const int quality = (int)config->quality;
const int low_effort = (config->method == 0); const int low_effort = (config->method == 0);
@ -1471,8 +1470,8 @@ WebPEncodingError VP8LEncodeStream(const WebPConfig* const config,
// Encode and write the transformed image. // Encode and write the transformed image.
err = EncodeImageInternal(bw, enc->argb_, &enc->hash_chain_, enc->refs_, err = EncodeImageInternal(bw, enc->argb_, &enc->hash_chain_, enc->refs_,
enc->current_width_, height, quality, low_effort, enc->current_width_, height, quality, low_effort,
use_cache, &enc->cache_bits_, enc->histo_bits_, &enc->cache_bits_, enc->histo_bits_, byte_position,
byte_position, &hdr_size, &data_size); &hdr_size, &data_size);
if (err != VP8_ENC_OK) goto Error; if (err != VP8_ENC_OK) goto Error;
if (picture->stats != NULL) { if (picture->stats != NULL) {
@ -1557,7 +1556,7 @@ int VP8LEncodeImage(const WebPConfig* const config,
if (!WebPReportProgress(picture, 5, &percent)) goto UserAbort; if (!WebPReportProgress(picture, 5, &percent)) goto UserAbort;
// Encode main image stream. // Encode main image stream.
err = VP8LEncodeStream(config, picture, &bw, 1 /*use_cache*/); err = VP8LEncodeStream(config, picture, &bw);
if (err != VP8_ENC_OK) goto Error; if (err != VP8_ENC_OK) goto Error;
// TODO(skal): have a fine-grained progress report in VP8LEncodeStream(). // TODO(skal): have a fine-grained progress report in VP8LEncodeStream().

3
src/enc/vp8li.h
View File

@ -64,10 +64,9 @@ int VP8LEncodeImage(const WebPConfig* const config,
const WebPPicture* const picture); const WebPPicture* const picture);
// Encodes the main image stream using the supplied bit writer. // Encodes the main image stream using the supplied bit writer.
// If 'use_cache' is false, disables the use of color cache.
WebPEncodingError VP8LEncodeStream(const WebPConfig* const config, WebPEncodingError VP8LEncodeStream(const WebPConfig* const config,
const WebPPicture* const picture, const WebPPicture* const picture,
VP8LBitWriter* const bw, int use_cache); VP8LBitWriter* const bw);
//------------------------------------------------------------------------------ //------------------------------------------------------------------------------