id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc
1393	Unnecessary condition checking when handling no output of prior picture	fhendry		"There is the following checking within function checkNoOutputPriorPics
""if (m_lastPOCNoOutputPriorPics != pcPicTmp->getPOC())""

This checking is unnecessary and cause a bug for the following cases:
Suppose that original bitstream coded with random access structure with IDR at every 32 pictures and the second IDR has its value of no_output_of_prior_pics_flag equal to 1. Then all other non-IDR pictures is removed from the bitstream leaving the bitstream.

It is expected that when decoder shall not output the first IDR because after the first IDR has been decoded (and gets into the DPB) it will be flushed without outputting it because the next IDR has no_output_of_prior_pics_flag equal to 1. However, because of the above checking, decoder will not mark the first IDR as not to output because its POC is the same as the POC of the second IDR.

Simply removing that condition would fix the bug.
Patch for this removal is attached."	defect	new	minor		HM	HM-16.4			ksuehring davidf karlsharman jct-vc@…