Commit b6ec6930 authored by Jingning Han's avatar Jingning Han Committed by Gerrit Code Review
Browse files

Remove redundant buffer initialization and mode_info assignments

There is no need to initialize source/dst frame buffers at frame
level. These will be done at block coding stage. This commit hence
removes the redundant operations.

Change-Id: I11d9f2556058c6205c8e58ed53e31f78622c41b7
parent ef2f6dae
...@@ -1330,8 +1330,10 @@ static int is_background(VP9_COMP *cpi, const TileInfo *const tile, ...@@ -1330,8 +1330,10 @@ static int is_background(VP9_COMP *cpi, const TileInfo *const tile,
int threshold = 0; int threshold = 0;
vp9_setup_src_planes(x, cpi->Source, mi_row, mi_col); vp9_setup_src_planes(x, cpi->Source, mi_row, mi_col);
src_stride = x->plane[0].src.stride; src_stride = x->plane[0].src.stride;
src = x->plane[0].src.buf; src = x->plane[0].src.buf;
pre_stride = cpi->Last_Source->y_stride; pre_stride = cpi->Last_Source->y_stride;
pre = cpi->Last_Source->y_buffer + (mi_row * MI_SIZE) * pre_stride + pre = cpi->Last_Source->y_buffer + (mi_row * MI_SIZE) * pre_stride +
(mi_col * MI_SIZE); (mi_col * MI_SIZE);
...@@ -2375,16 +2377,8 @@ static void init_encode_frame_mb_context(VP9_COMP *cpi) { ...@@ -2375,16 +2377,8 @@ static void init_encode_frame_mb_context(VP9_COMP *cpi) {
// Copy data over into macro block data structures. // Copy data over into macro block data structures.
vp9_setup_src_planes(x, cpi->Source, 0, 0); vp9_setup_src_planes(x, cpi->Source, 0, 0);
// TODO(jkoleszar): are these initializations required?
vp9_setup_pre_planes(xd, 0, get_ref_frame_buffer(cpi, LAST_FRAME), 0, 0,
NULL);
vp9_setup_dst_planes(xd, get_frame_new_buffer(cm), 0, 0);
vp9_setup_block_planes(&x->e_mbd, cm->subsampling_x, cm->subsampling_y); vp9_setup_block_planes(&x->e_mbd, cm->subsampling_x, cm->subsampling_y);
xd->mi[0]->mbmi.mode = DC_PRED;
xd->mi[0]->mbmi.uv_mode = DC_PRED;
// Note: this memset assumes above_context[0], [1] and [2] // Note: this memset assumes above_context[0], [1] and [2]
// are allocated as part of the same buffer. // are allocated as part of the same buffer.
vpx_memset(xd->above_context[0], 0, vpx_memset(xd->above_context[0], 0,
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment