From 09858c239bd75a76a078027f3921315509340956 Mon Sep 17 00:00:00 2001 From: Dmitry Kovalev <dkovalev@google.com> Date: Thu, 22 Aug 2013 14:39:05 -0700 Subject: [PATCH] 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 --- vp9/decoder/vp9_decodframe.c | 4 ---- 1 file changed, 4 deletions(-) diff --git a/vp9/decoder/vp9_decodframe.c b/vp9/decoder/vp9_decodframe.c index 70e85f95a1..f65ef04e87 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); -- GitLab