Opened 11 years ago

Closed 9 years ago

#891 closed enhancement (fixed)

[clean-ups] Merge-related codet

Reported by: huiyong Owned by:
Priority: minor Milestone: HM-16.3
Component: HM Version: HM-9.1
Keywords: Cc: fbossen, ksuehring, davidf, jct-vc@…

Description

For better understanding of the merge-related code part, I suggest the following clean-ups:
(1) Removal of unused arguments of decodeMergeIndex(): puhInterDirNeighbours and pcMvFieldNeighbours.
(2) Change of the name of variable isMerged to hasMergeCandList for better understanding.
(3) Prevention of unnecessary (i.e., has-no-effect) calls of pcSubCU->setPartSizeSubParts() both in encoder and decoder.

Please check the attached patch file for details.
Note that the suggested clean-ups have no effect on the encoded and decoded results.

Attachments (3)

Cleanup-decodePUWise().zip (1.2 KB) - added by huiyong 11 years ago.
Cleanup-decodePUWise()_HM10.0.zip (914 bytes) - added by huiyong 11 years ago.
clean-up patch for the released HM10.0 version
Cleanup-decodePUWise()_HM10.0-dev@3363.zip (932 bytes) - added by huiyong 11 years ago.
clean-up patch for a more recent branch version (dev@3363)

Download all attachments as: .zip

Change History (6)

comment:1 Changed 11 years ago by DefaultCC Plugin

  • Cc fbossen ksuehring davidf jct-vc@… added

Changed 11 years ago by huiyong

Changed 11 years ago by huiyong

clean-up patch for the released HM10.0 version

Changed 11 years ago by huiyong

clean-up patch for a more recent branch version (dev@3363)

comment:2 Changed 10 years ago by ksuehring

  • Milestone HM-9.2 deleted

comment:3 Changed 9 years ago by karlsharman

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

Applied in r4241.

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)
  • Hui Yong Kim(Reporter)
  • jct-vc@…(Subscriber)
  • Karl Sharman(Participant)
  • karl.sharman@…(Always)
  • Karsten Suehring(Subscriber, Participant, Always)