From: Dmitry Kovalev Date: Thu, 22 Aug 2013 21:39:05 +0000 (-0700) Subject: Removing useless calls to setup_{pre, dst}_planes. X-Git-Tag: v1.3.0~540^2 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=09858c239bd75a76a078027f3921315509340956;p=libvpx Removing useless calls to setup_{pre, dst}_planes. Comment is wrong, we don't initialize any xd pointers. We only initialize xd->planes[i]->dst and xd->planes[i]->pre[], which are actually initialized for every block during the decoding. Change-Id: If152ea872ebef1f83ca70712fa6f8df1b6855f56 --- diff --git a/vp9/decoder/vp9_decodframe.c b/vp9/decoder/vp9_decodframe.c index 70e85f95a..f65ef04e8 100644 --- a/vp9/decoder/vp9_decodframe.c +++ b/vp9/decoder/vp9_decodframe.c @@ -969,10 +969,6 @@ int vp9_decode_frame(VP9D_COMP *pbi, const uint8_t **p_data_end) { vp9_zero(pc->counts); - // Initialize xd pointers. Any reference should do for xd->pre, so use 0. - setup_pre_planes(xd, 0, &pc->yv12_fb[pc->active_ref_idx[0]], 0, 0, NULL); - setup_dst_planes(xd, new_fb, 0, 0); - new_fb->corrupted |= read_compressed_header(pbi, data, first_partition_size); setup_block_dptrs(xd, pc->subsampling_x, pc->subsampling_y);