Merge "Removing gold_is_last, alt_is_last, gold_is_alt flags."

This commit is contained in:
Dmitry Kovalev 2014-08-20 12:10:14 -07:00 committed by Gerrit Code Review
commit c8e933ef4b
2 changed files with 16 additions and 34 deletions

View File

@ -715,11 +715,6 @@ VP9_COMP *vp9_create_compressor(VP9EncoderConfig *oxcf) {
vp9_rc_init(&cpi->oxcf, oxcf->pass, &cpi->rc); vp9_rc_init(&cpi->oxcf, oxcf->pass, &cpi->rc);
cm->current_video_frame = 0; cm->current_video_frame = 0;
cpi->gold_is_last = 0;
cpi->alt_is_last = 0;
cpi->gold_is_alt = 0;
cpi->skippable_frame = 0; cpi->skippable_frame = 0;
// Create the encoder segmentation map and set all entries to 0 // Create the encoder segmentation map and set all entries to 0
@ -1900,36 +1895,27 @@ static void encode_with_recode_loop(VP9_COMP *cpi,
} while (loop); } while (loop);
} }
static void get_ref_frame_flags(VP9_COMP *cpi) { static int get_ref_frame_flags(const VP9_COMP *cpi) {
if (cpi->refresh_last_frame & cpi->refresh_golden_frame) const int *const map = cpi->common.ref_frame_map;
cpi->gold_is_last = 1; const int gold_is_last = map[cpi->gld_fb_idx] == map[cpi->lst_fb_idx];
else if (cpi->refresh_last_frame ^ cpi->refresh_golden_frame) const int alt_is_last = map[cpi->alt_fb_idx] == map[cpi->lst_fb_idx];
cpi->gold_is_last = 0; const int gold_is_alt = map[cpi->gld_fb_idx] == map[cpi->alt_fb_idx];
int flags = VP9_ALT_FLAG | VP9_GOLD_FLAG | VP9_LAST_FLAG;
if (cpi->refresh_last_frame & cpi->refresh_alt_ref_frame) if (gold_is_last)
cpi->alt_is_last = 1; flags &= ~VP9_GOLD_FLAG;
else if (cpi->refresh_last_frame ^ cpi->refresh_alt_ref_frame)
cpi->alt_is_last = 0;
if (cpi->refresh_alt_ref_frame & cpi->refresh_golden_frame)
cpi->gold_is_alt = 1;
else if (cpi->refresh_alt_ref_frame ^ cpi->refresh_golden_frame)
cpi->gold_is_alt = 0;
cpi->ref_frame_flags = VP9_ALT_FLAG | VP9_GOLD_FLAG | VP9_LAST_FLAG;
if (cpi->gold_is_last)
cpi->ref_frame_flags &= ~VP9_GOLD_FLAG;
if (cpi->rc.frames_till_gf_update_due == INT_MAX && if (cpi->rc.frames_till_gf_update_due == INT_MAX &&
!is_spatial_svc(cpi)) !is_spatial_svc(cpi))
cpi->ref_frame_flags &= ~VP9_GOLD_FLAG; flags &= ~VP9_GOLD_FLAG;
if (cpi->alt_is_last) if (alt_is_last)
cpi->ref_frame_flags &= ~VP9_ALT_FLAG; flags &= ~VP9_ALT_FLAG;
if (cpi->gold_is_alt) if (gold_is_alt)
cpi->ref_frame_flags &= ~VP9_ALT_FLAG; flags &= ~VP9_ALT_FLAG;
return flags;
} }
static void set_ext_overrides(VP9_COMP *cpi) { static void set_ext_overrides(VP9_COMP *cpi) {
@ -2236,7 +2222,7 @@ static void encode_frame_to_data_rate(VP9_COMP *cpi,
else else
cpi->frame_flags &= ~FRAMEFLAGS_ALTREF; cpi->frame_flags &= ~FRAMEFLAGS_ALTREF;
get_ref_frame_flags(cpi); cpi->ref_frame_flags = get_ref_frame_flags(cpi);
cm->last_frame_type = cm->frame_type; cm->last_frame_type = cm->frame_type;
vp9_rc_postencode_update(cpi, *size); vp9_rc_postencode_update(cpi, *size);

View File

@ -244,10 +244,6 @@ typedef struct VP9_COMP {
YV12_BUFFER_CONFIG *unscaled_last_source; YV12_BUFFER_CONFIG *unscaled_last_source;
YV12_BUFFER_CONFIG scaled_last_source; YV12_BUFFER_CONFIG scaled_last_source;
int gold_is_last; // gold same as last frame ( short circuit gold searches)
int alt_is_last; // Alt same as last ( short circuit altref search)
int gold_is_alt; // don't do both alt and gold search ( just do gold).
int skippable_frame; int skippable_frame;
int scaled_ref_idx[3]; int scaled_ref_idx[3];