Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#7 closed defect (fixed)

missing a close-bracket on slice_pic_order_cnt_lsb

Reported by: tsukuba.takeshi Owned by: jlchen
Priority: trivial Milestone:
Component: SHVC text and test model Version:
Keywords: Cc: jlchen, yanye, jillboyce, jct-vc@…

Description

A close-bracket corresponding to a if statement, i.e.,
if( nuh_layer_id > 0 | | ( nal_unit_type != IDR_W_RADL && nal_unit_type != IDR_N_LP ) ) is missed.

There are two options.

Op.1) Insert a close-bracket after slice_temporal_mvp_enabled_flag
Op.2) Remove a open-bracket

In SHM, currently Op.1 was already integrated. Those changes don’t affect decoding/encoding process at all.

/ quote from O1008_v1 /
if( ( nuh_layer_id > 0 ) | | ( nal_unit_type != IDR_W_RADL && nal_unit_type != IDR_N_LP ) ) "{" / Op.2) remove this open-bracket /

slice_pic_order_cnt_lsb

if( nal_unit_type != IDR_W_RADL && nal_unit_type != IDR_N_LP ) {

short_term_ref_pic_set_sps_flag
…
if( sps_temporal_mvp_enabled_flag )

slice_temporal_mvp_enabled_flag

}
"}" / Op.1) insert a close-bracket here /

Change history (4)

comment:1 Changed 11 years ago by DefaultCC Plugin

  • Cc jlchen yanye jillboyce jct-vc@… added

comment:2 Changed 11 years ago by jlchen

It looks that op.1 and op.2 have the same effect. I will take op.2 to fix the text since it looks clearer to me.

Thanks,

comment:3 Changed 11 years ago by jlchen

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

comment:4 Changed 11 years ago by jlchen

  • Milestone Draft Text 4 deleted

Milestone Draft Text 4 deleted

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

  • Gerhard Tech(Always)
  • jct-vc@…(Subscriber)
  • Jianle Chen(Owner, Subscriber, Participant)
  • Jill Boyce(Subscriber)
  • Karsten Suehring(Always)
  • Takeshi Tsukuba(Reporter)
  • yanye(Subscriber)