Opened 11 years ago

Closed 11 years ago

#1113 closed defect (fixed)

Incorrect handling of mismatch between reference pictures and reference idcs

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

Description

There is the following code fragment in the file TEncTop.cpp near the line 800:

if (numNeg != rps->getNumberOfNegativePictures())
{

printf("Warning: number of negative pictures in RPS is different between intra and inter RPS specified in the config file.\n");
rps->setNumberOfNegativePictures(numNeg);
rps->setNumberOfPositivePictures(numNeg+numPos);

}
if (numPos != rps->getNumberOfPositivePictures())
{

printf("Warning: number of positive pictures in RPS is different between intra and inter RPS specified in the config file.\n");
rps->setNumberOfPositivePictures(numPos);
rps->setNumberOfPositivePictures(numNeg+numPos);

}

Looks like it should be:

if (numNeg != rps->getNumberOfNegativePictures())
{

printf("Warning: number of negative pictures in RPS is different between intra and inter RPS specified in the config file.\n");
rps->setNumberOfNegativePictures(numNeg);
rps->setNumberOfPictures(numNeg+numPos);

}
if (numPos != rps->getNumberOfPositivePictures())
{

printf("Warning: number of positive pictures in RPS is different between intra and inter RPS specified in the config file.\n");
rps->setNumberOfPositivePictures(numPos);
rps->setNumberOfPictures(numNeg+numPos);

}

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

Fixed in r3520

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

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