Opened 4 years ago

Closed 4 years ago

#1150 closed defect (fixed)

Typo for presenceFlag in picture header syntax

Reported by: martin.m.pettersson Owned by:
Priority: minor Milestone:
Component: spec Version: VVC D9 vB
Keywords: Cc: ksuehring, bbross, XiangLi, fbossen, jvet@…

Description

JVET-S2001-v3 has the following text in the picture header syntax:

presenceFlag = 0
if( !pps_rpl_info_in_ph_flag ) /* This condition is intentionally not merged into the next, to avoid possible intepretation of RplsIdx[ i ] being unspecified. */

presenceFlag = 1
else if( num_ref_entries[ 1 ][ RplsIdx[ 1 ] ] > 0 )

presenceFlag = 1

if( presenceFlag! ) {

ph_mvd_l1_zero_flag u(1)
if( sps_bdof_control_present_in_ph_flag )

ph_bdof_disabled_flag u(1)

if( sps_dmvr_control_present_in_ph_flag )

ph_dmvr_disabled_flag u(1)

}

The line "if( presenceFlag! ) {" seems to have a typo with an exclamation mark that should not be there. It is suggested to change it to "if( presenceFlag ) {".

Maybe the name of the presenceFlag could also be changed to something more meaningful such as bipredToolControlPresentFlag.

Change history (1)

comment:1 Changed 4 years ago by bbross

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

Good catch, will be fixed in JVET-S2001-v7.

Note: See TracTickets for help on using tickets.