Opened 11 years ago

Closed 11 years ago

#906 closed defect (fixed)

Mismatch in location of enable_temporal_mvp_flag

Reported by: peterderivaz Owned by: bbross
Priority: minor Milestone:
Component: Text Version: D9 (K1003) v11
Keywords: Cc: bbross, wjhan, jct-vc@…

Description

In 7.3.8.1 "General slice segment header syntax", the text has slice_temporal_mvp_enable_flag appearing just before slice_sao_luma_flag, but the reference code 9.1rc1 has it appearing afterwards.

if( sps_temporal_mvp_enable_flag )

slice_temporal_mvp_enable_flag

}
if( sample_adaptive_offset_enabled_flag ) {

slice_sao_luma_flag
slice_sao_chroma_flag

}

Line 1054 of TDecCAVLC.cpp:

READ_FLAG(uiCode, "slice_sao_luma_flag"); rpcSlice->setSaoEnabledFlag((Bool)uiCode);


Line 1069:

READ_FLAG( uiCode, "enable_temporal_mvp_flag" );


Change History (2)

comment:1 Changed 11 years ago by DefaultCC Plugin

  • Cc bbross wjhan jct-vc@… added

comment:2 Changed 11 years ago by bbross

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

This mismatch was fixed in the current HM-9.1-dev branch r3201 and r3209.

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

  • Benjamin Bross(Owner, Subscriber, Participant)
  • jct-vc@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Always)
  • Peter de Rivaz(Reporter)
  • Woo-Jin Han(Subscriber)