Opened 9 years ago

Closed 9 years ago

#545 closed defect (fixed)

Typo in picture padding/cropping.

Reported by: bheng Owned by:
Priority: minor Milestone: HM-7.1
Component: HM Version: HM-7.0
Keywords: Cc: fbossen, ksuehring, davidf, jct-vc@…

Description

There is a typo in the following section of TAppEncCfg::parseCfg( ) that is requiring 4:2:0 picture heights to have an odd number of luma samples instead of preventing that from happening.

if (m_aiPad[1] % TComSPS::getCropUnitY(CHROMA_420) != 1)
{
  fprintf(stderr, "Error: picture height is not an integer multiple of the specified chroma subsampling\n");
  exit(EXIT_FAILURE);
}

The "!= 1" should be "!= 0".

Change History (2)

comment:1 Changed 9 years ago by DefaultCC Plugin

  • Cc fbossen ksuehring davidf jct-vc@… added

comment:2 Changed 9 years ago by ksuehring

  • Milestone set to HM-7.1
  • Resolution set to fixed
  • Status changed from new to closed

fixed in r2413

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

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