Opened 10 years ago

Closed 10 years ago

#168 closed defect (fixed)

JCTVC-E049 bug about the CABAC memory allocation for SAO

Reported by: chihming.fu Owned by:
Priority: minor Milestone: HM-3.1
Component: HM Version: HM-3.0
Keywords: Cc: fbossen, ksuehring, davidf, jct-vc@…

Description

Using revision r929
In TEncAdaptiveLoopFilter.cpp:4984, add variables for
SAO encoder without using global variable m_pppcRDSbacCoder.

  Int iMaxDepth = 4;
  m_pppcRDSbacCoder = new TEncSbac** [iMaxDepth+1];
  m_pppcBinCoderCABAC = new TEncBinCABAC** [iMaxDepth+1];

  for ( Int iDepth = 0; iDepth < iMaxDepth+1; iDepth++ )
  {
    m_pppcRDSbacCoder[iDepth] = new TEncSbac* [CI_NUM];
    m_pppcBinCoderCABAC[iDepth] = new TEncBinCABAC* [CI_NUM];

    for (Int iCIIdx = 0; iCIIdx < CI_NUM; iCIIdx ++ )
    {
      m_pppcRDSbacCoder[iDepth][iCIIdx] = new TEncSbac;
      m_pppcBinCoderCABAC [iDepth][iCIIdx] = new TEncBinCABAC;
      m_pppcRDSbacCoder   [iDepth][iCIIdx]->init( m_pppcBinCoderCABAC [iDepth][iCIIdx] );
    }
  }

Also, these variables should be deleted in function destoryEncBuffer().

Change History (4)

comment:1 Changed 10 years ago by fbossen

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

Fixed in r957

comment:2 Changed 10 years ago by davidf

  • Component set to HM

Updating component after adding WD (Text) tickets

comment:3 Changed 9 years ago by davidf

  • Cc fbossen ksuehring davidf added

comment:4 Changed 9 years ago by davidf

  • Cc jct-vc@… added
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

  • Chih-Ming Fu(Reporter)
  • David Flynn(Subscriber, Participant)
  • Frank Bossen(Subscriber, Participant)
  • jct-vc@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Subscriber, Always)