dany/libwebp
1
0
Fork 0
mirror of https://github.com/webmproject/libwebp.git synced 2025-02-13 07:22:52 +01:00
Code Issues Actions Packages Projects Releases Wiki Activity

AnimEncoder API: Fix for kmax=1 and default kmin case.

Browse Source 
Some frames that were previously selected as key-frames were incorrectly
being reset to sub-frames.

Change-Id: Iee342dbb9a9aec144b8185c3b54ca56aa7038bfb
This commit is contained in:
Urvang Joshi 2015-01-29 13:26:01 -08:00
parent 022d2f886c
commit c6b24543fc
1 changed files with 4 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
src/mux/anim_encode.c
View File

@ -975,6 +975,8 @@ static int CacheFrame(WebPAnimEncoder* const enc, int duration,
if (enc->count_since_key_frame_ == enc->options_.kmax) {
enc->flush_count_ = enc->count_ - 1;
enc->count_since_key_frame_ = 0;
enc->keyframe_ = KEYFRAME_NONE;
enc->best_delta_ = DELTA_INFINITY;
}
enc->prev_candidate_undecided_ = 1;
}
@ -1001,20 +1003,10 @@ static int CacheFrame(WebPAnimEncoder* const enc, int duration,
static int FlushFrames(WebPAnimEncoder* const enc) {
while (enc->flush_count_ > 0) {
WebPMuxFrameInfo* info;
WebPMuxError err;
EncodedFrame* const curr = GetFrame(enc, 0);
// Pick frame or full canvas.
if (curr->is_key_frame_) {
info = &curr->key_frame_;
if (enc->keyframe_ == 0) {
enc->keyframe_ = KEYFRAME_NONE;
enc->best_delta_ = DELTA_INFINITY;
}
} else {
info = &curr->sub_frame_;
}
// Add to mux.
const WebPMuxFrameInfo* const info =
curr->is_key_frame_ ? &curr->key_frame_ : &curr->sub_frame_;
assert(enc->mux_ != NULL);
err = WebPMuxPushFrame(enc->mux_, info, 1);
if (err != WEBP_MUX_OK) {