Opened 9 years ago

Closed 9 years ago

#532 closed defect (fixed)

Can not parse "FrameToBeEncoded" in encoding config file correctly in HM-6.3

Reported by: peterchuang Owned by:
Priority: major Milestone: HM-7.0
Component: HM Version: HM-6.3
Keywords: Cc: fbossen, ksuehring, davidf, jct-vc@…

Description

In HM-6.3, TAppEncCfg.cpp, line 197,
the "FramesToBeEncoded" should be "FrameToBeEncoded" (should not have a extra "s").

Change History (6)

comment:1 Changed 9 years ago by DefaultCC Plugin

  • Cc fbossen ksuehring davidf jct-vc@… added

comment:2 Changed 9 years ago by ksuehring

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

The parameter without the 's' has been deprecated since HM 1 or 2. It has been removed now.
FramesToBeEncoded is aligned with JM and more suitable because this describes a number of frame, not a property of a single frame.

comment:3 Changed 9 years ago by peterchuang

Then, I would like to suggest to modify the cfg files. Change the "FrameToBeEncoded" to "FramesToBeEncoded".

comment:4 Changed 9 years ago by ksuehring

  • Resolution wontfix deleted
  • Status changed from closed to reopened

OK. I missed that we still have the old parameter in the file. Reopening.

comment:5 Changed 9 years ago by ksuehring

  • Milestone changed from HM-6.3 to HM-7.0
  • Version changed from HM-6.1 to HM-6.3

comment:6 Changed 9 years ago by ksuehring

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

Fixed in HM-6.3.1 and HM-6.3-dev

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