Opened 11 years ago

Closed 11 years ago

#236 closed defect (fixed)

Tiles parameter naming convention change

Reported by: shilin.xu Owned by:
Priority: minor Milestone:
Component: HM Version: HM-4.1
Keywords: Cc: fbossen, ksuehring, davidf, jct-vc@…


The most recent WD text change two tiles parameters from "tile_boundary_independence_idc" and "uniform_spacing_idc" to "tile_boundary_independence_flag" and "uniform_spacing_flag", so we would like to make the same change in HM code to correspond to the WD test.
The places of the HM code which should be changed are:

1) In codePPS() function of TEncCavlc.cpp file,
WRITE_FLAG( pcPPS->getUniformSpacingIdr(), "uniform_spacing_idc" );
WRITE_FLAG( pcPPS->getTileBoundaryIndependenceIdr(), "tile_boundary_independence_idc" );
2) In codeSPS() function of TEncCavlc.cpp file,
WRITE_FLAG( pcSPS->getUniformSpacingIdr(), "uniform_spacing_idc" );
WRITE_FLAG( pcSPS->getTileBoundaryIndependenceIdr(), "tile_boundary_independence_idc" );
3) In parsePPS() function of TDecCAVLC.cpp file,
READ_FLAG ( uiCode, "uniform_spacing_idc" );
READ_FLAG ( uiCode, "tile_boundary_independence_idc" );
4) In parseSPS() function of TDecCAVLC,cpp file,
READ_FLAG ( uiCode, "uniform_spacing_idc" );
READ_FLAG ( uiCode, "tile_boundary_independence_idc" );

What we would like to do is change the "idc" to "flag".

Change History (4)

comment:1 Changed 11 years ago by DefaultCC Plugin

  • Cc fbossen ksuehring davidf added

comment:2 Changed 11 years ago by davidf

Updating component after adding WD (Text) tickets

comment:3 Changed 11 years ago by fbossen

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

Fixed in r1690

comment:4 Changed 10 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, Always)
  • Shilin Xu(Reporter)