Opened 11 years ago

Closed 11 years ago

#1030 closed defect (fixed)

Buffer size in HM decoder

Reported by: adarsh Owned by:
Priority: minor Milestone: HM-11.0
Component: HM Version: HM-10.0
Keywords: Cc: fbossen, ksuehring, davidf, jct-vc@…

Description

In TDecTop.cpp, there is a line as follows.

m_iMaxRefPicNum = pcSlice->getSPS()->getMaxDecPicBuffering(pcSlice->getTLayer())+pcSlice->getSPS()->getNumReorderPics(pcSlice->getTLayer()) + 1;

This variable m_iMaxRefPicNum is used to compare with the size of the variable m_cListPic . Why should num. of reordered pictures be part of the calculation? Shouldn't it just be as follows?

m_iMaxRefPicNum = pcSlice->getSPS()->getMaxDecPicBuffering(pcSlice->getTLayer()) + 1;

Also, it might be helpful to rename the variable m_iMaxRefPicNum because it is not indicating the number of reference pictures.

Change History (2)

comment:1 Changed 11 years ago by DefaultCC Plugin

  • Cc fbossen ksuehring davidf jct-vc@… added

comment:2 Changed 11 years ago by fbossen

  • Milestone set to HM-11.0
  • Resolution set to fixed
  • Status changed from new to closed

Fixed in r3473

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

  • Adarsh Krishnan Ramasubramonian(Reporter)
  • David Flynn(Subscriber)
  • Frank Bossen(Subscriber, Participant)
  • jct-vc@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Subscriber, Always)