id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc
1215	Minor improvement in clarifying the branch condition for parsing rqt_root_cbf flag	cheerchen73		"In JCTVC-O1005_v2:

coding_quadtree( x0, y0, log2CbSize, cqtDepth ) {
...
if( CuPredMode[ x0 ][ y0 ]  !=  MODE_INTRA  '''&&'''  
    !( PartMode  = =  PART_2Nx2N  &&  merge_flag[ x0 ][ y0 ] )  '''| |'''  
    CuPredMode[ x0 ][ y0 ]  = =  MODE_INTRA  &&  intra_bc_flag[ x0 ][ y0 ] )
    '''rqt_root_cbf'''
...
}

In the RExt text specification, the aforementioned branch condition looks kind of vague.
Someone may think the first '''&&''' is carried out prior to the '''| |''' while others may do it in reverse order, resulting in a very different consequence.

For example, if the '''| |''' are carried out prior to the '''&&''' and the prediction mode is IntraBC, the '''rqt_root_cbf''' will never be parsed.
To prevent from this, I would suggest revising the text as follows:

coding_quadtree( x0, y0, log2CbSize, cqtDepth ) {
...
if( '''(''' CuPredMode[ x0 ][ y0 ]  !=  MODE_INTRA  '''&&'''  
    !( PartMode  = =  PART_2Nx2N  &&  merge_flag[ x0 ][ y0 ] ) ''')'''  '''| |'''  
    CuPredMode[ x0 ][ y0 ]  = =  MODE_INTRA  &&  intra_bc_flag[ x0 ][ y0 ] )
    '''rqt_root_cbf'''
...
}"	defect	closed	minor	RExt D6	RExt text	RExt D5 (O1005) v2	fixed		davidf joel teruhiko jct-vc@…