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.