Opened 8 years ago

Closed 8 years ago

#1215 closed defect (fixed)

Minor improvement in clarifying the branch condition for parsing rqt_root_cbf flag

Reported by: cheerchen73 Owned by:
Priority: minor Milestone: RExt D6
Component: RExt text Version: RExt D5 (O1005) v2
Keywords: Cc: davidf, joel, teruhiko, jct-vc@…

Description

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

...
}

Change History (2)

comment:1 Changed 8 years ago by DefaultCC Plugin

  • Cc davidf joel teruhiko jct-vc@… added

comment:2 Changed 8 years ago by davidf

  • Resolution set to fixed
  • Status changed from new to closed

Addressed (fixed)

Note: See TracTickets for help on using tickets.

This list contains all users that will be notified about changes made to this ticket.

These roles will be notified: Reporter, Owner, Subscriber, Participant

  • Chun-Chi Chen(Reporter)
  • David Flynn(Subscriber, Participant)
  • jct-vc@…(Subscriber)
  • Joel(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Always)
  • Teruhiko Suzuki(Subscriber)