Changes between Initial Version and Version 1 of Ticket #1097, comment 5


Ignore:
Timestamp:
May 29, 2013, 11:39:46 AM (7 years ago)
Author:
jackh
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #1097, comment 5

    initial v1  
    2020}}}
    2121
    22 The problem seems to be this. When setPOC() is called in the section of TDecCAVLC.cpp we've been discussing, it passes just the LSB of the referenced picture, not the full POC - in order to do this it would have to do the DPB lookup. xGetLongTermRefPic() goes through the DPB and finds the picture correctly, but when getPOC() is returned on that picture it returns the LSB, because that's what was stored originally (239 in my example). This is then compared to the full POC for the previous CRA (751 in my example), and the assert fires.
     22The problem seems to be this. When setPOC() is called in the section of TDecCAVLC.cpp we've been discussing, it passes just the LSB of the referenced picture, not the full POC - in order to do this it would have to do the DPB lookup. xGetLongTermRefPic() goes through the DPB and finds the picture correctly, but when getPOC() is called on that picture it returns the LSB, because that's what was stored originally (239 in my example). This is then compared to the full POC for the previous CRA (751 in my example), and the assert fires.
    2323
    2424The solution seems to be to do the lookup in TDecCAVLC, and call setPOC and setDeltaPOC according to the full POC found in the DPB.

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(Participant)
  • David Flynn(Subscriber)
  • Frank Bossen(Subscriber, Participant)
  • Jack Haughton(Reporter, Participant)
  • jct-vc@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Subscriber, Participant, Always)