Opened 13 years ago

Closed 13 years ago

#242 closed defect (fixed)

ListCombination config parameter verification was disabled when the LowDelayCoding parameter was removed.

Reported by: tktan Owned by:
Priority: minor Milestone: HM-5.1
Component: HM Version: HM-5.0rc1
Keywords: Cc: fbossen, ksuehring, davidf, jct-vc@…

Description

This defect changes the meaning of the ListCombination parameter and causes a mismatch with the WD text on the ref_pic_list_combination_flag.

A simple bug fix is to check the variable m_uiMaxNumberOfReorderPictures instead of the LowDelayCoding parameter that is no longer there.

See #if LC_BUGFIX in attached patch to HM-5.0rc1.

Two minor issues are also fixed in the attached patch.
1) comments for the following variables were swapped around in TAppEngCfg.h

UInt m_uiMaxNumberOfReorderPictures;
UInt m_uiMaxNumberOfReferencePictures;

2) the string "ListCombination, -lc" should not have a " " before "-lc"

Attachments (1)

ListCOmbination_bugfix.patch​ (2.9 KB) - added by tktan 13 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 13 years ago by DefaultCC Plugin

  • Cc fbossen ksuehring davidf added

Changed 13 years ago by tktan

comment:2 Changed 13 years ago by ksuehring

  • Milestone set to HM-5.1

comment:3 Changed 13 years ago by fbossen

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

Fixed in r1689

comment:4 Changed 13 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

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