Opened 10 years ago Closed 10 years ago #57 closed defect (fixed)Setting the PTL in the init() function
Description
Old way: Void TEncTop::init(Bool isFieldCoding) { // initialize SPS xInitSPS(); /* set the VPS profile information */ *m_cVPS.getPTL() = *m_cSPS.getPTL();
After rev 941 (and rev 976) fix, the PTL changes are in. if MULTIPLE_PTL_SUPPORT //Populate PTL in VPS TComVPS *pVPS = m_acTEncTop[0].getVPS(); for (int ii = 0; ii < m_numPTLInfo; ii++) { pVPS->getPTL(ii)->getGeneralPTL()->setLevelIdc(m_levelList[ii]); pVPS->getPTL(ii)->getGeneralPTL()->setTierFlag(m_levelTierList[ii]); pVPS->getPTL(ii)->getGeneralPTL()->setProfileIdc(m_profileList[ii]); pVPS->getPTL(ii)->getGeneralPTL()->setProfileCompatibilityFlag(m_profileCompatibility[ii], 1); pVPS->getPTL(ii)->getGeneralPTL()->setProgressiveSourceFlag(m_progressiveSourceFlagList[ii]); pVPS->getPTL(ii)->getGeneralPTL()->setInterlacedSourceFlag(m_interlacedSourceFlagList[ii]); pVPS->getPTL(ii)->getGeneralPTL()->setNonPackedConstraintFlag(m_nonPackedConstraintFlagList[ii]); pVPS->getPTL(ii)->getGeneralPTL()->setFrameOnlyConstraintFlag(m_frameOnlyConstraintFlagList[ii]); }
This line could be commented / removed. /* set the VPS profile information */ //*m_cVPS.getPTL() = *m_cSPS.getPTL(); Attachments (1)Change history (4)comment:1 Changed 10 years ago by DefaultCC Plugin
Changed 10 years ago by fhendrycomment:2 in reply to: ↑ description Changed 10 years ago by fhendrycomment:3 Changed 10 years ago by Vadim
the suggested line was disabled with rev 1017, please reopen the ticket if the problem still exists 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
|
Agree that line should be removed.
In addition, some asserts should be added to enable the software to notify if incorrect ptl value (violating conformance constraints) is used.
Replying to jaypadia: