Opened 7 years ago

Closed 7 years ago

#1307 closed defect (fixed)

Sizes of m_ltRefPicPocLsbSps and m_usedByCurrPicLtSPSFlag are incorrect

Reported by: jackh Owned by:
Priority: minor Milestone: HM+RExt-8.0
Component: HM Version: RExt-7.2 (HM-14.0)
Keywords: Cc: fbossen, ksuehring, davidf, jct-vc@…

Description

In HM-15.0, TComSlice.h contains these two definitions:

UInt m_ltRefPicPocLsbSps[33];
Bool m_usedByCurrPicLtSPSFlag[33];

In HM-15.0+RExt-8.0, they are incorrectly defined as:

UInt m_ltRefPicPocLsbSps[MAX_NUM_SPS + 1];
Bool m_usedByCurrPicLtSPSFlag[MAX_NUM_SPS + 1];

From the spec it looks as though these arrays can be as large as num_long_term_ref_pics_sps, so the correct size for these arrays would be 32?

There are also a few incorrect asserts relating to these definitions.

Change History (3)

comment:1 Changed 7 years ago by DefaultCC Plugin

  • Cc fbossen ksuehring davidf jct-vc@… added

comment:2 Changed 7 years ago by karlsharman

Fix applied in r4068.

comment:3 Changed 7 years ago by karlsharman

  • Resolution set to fixed
  • Status changed from new to closed
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

  • David Flynn(Subscriber)
  • Frank Bossen(Subscriber)
  • Jack Haughton(Reporter)
  • jct-vc@…(Subscriber)
  • Karl Sharman(Participant)
  • karl.sharman@…(Always)
  • Karsten Suehring(Subscriber, Always)