Opened 8 years ago

Closed 7 years ago

Last modified 7 years ago

#1118 closed defect (fixed)

initPattern uses wrong pixel offsets for boundary calculations, but results are still correct

Reported by: gordon Owned by:
Priority: trivial Milestone: HM-16.1
Component: HM Version: HM-11.0
Keywords: Cc: fbossen, ksuehring, davidf, jct-vc@…

Description

In TComPattern::initPattern, pcCU->getCUPelX() and Y are used in calculations to determine picture boundaries, but something like pcCU->getPic()->getCU( pcCU->getAddr())->getCUPelX() is the correct thing to use. (A CTU based g_auiRasterToPelX offset is being added to this value.)

It seems that the uiOffsetLeft and uiOffsetAbove calculation results don't change with this modification, as we're only interested in determining if we're at picture boundaries which occurs only when everything is zero.

Change History (3)

comment:1 Changed 8 years ago by DefaultCC Plugin

  • Cc fbossen ksuehring davidf jct-vc@… added

comment:2 Changed 7 years ago by karlsharman

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

Also see #296.
Code not present due to merger with RExt.

comment:3 Changed 7 years ago by ksuehring

  • Milestone set to HM-16.1
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)
  • gordon clare(Reporter)
  • jct-vc@…(Subscriber)
  • Karl Sharman(Participant)
  • karl.sharman@…(Always)
  • Karsten Suehring(Subscriber, Participant, Always)