Opened 6 years ago

Closed 6 years ago

Last modified 5 years ago

#1443 closed enhancement (fixed)

extra condition on reading of loop_filter_across_tiles_enabled_flag

Reported by: kolya Owned by:
Priority: minor Milestone: HM-16.10
Component: HM Version: HM-16.8
Keywords: Cc: ksuehring, davidf, karlsharman, jct-vc@…

Description

(on behalf of Pavel Evsikov pavel.evsikov@…)

In parsePPS, there is

if ((tileColumnsMinus1 + tileRowsMinus1) != 0)
{

READ_FLAG ( uiCode, "loop_filter_across_tiles_enabled_flag" ); pcPPS->setLoopFilterAcrossTilesEnabledFlag( uiCode ? true : false );

}

Though (tileColumnsMinus1 + tileRowsMinus1) != 0 is a conformance condition, the spec does not tell that reading of the flag is possible only when both reading parameters are not zeros. The

It is better to user "assert" for the conformance condition.

Change History (2)

comment:1 Changed 6 years ago by ksuehring

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

fixed in r4742

comment:2 Changed 5 years ago by ksuehring

  • Milestone changed from HM-next to HM-16.10
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)
  • jct-vc@…(Subscriber)
  • Karl Sharman(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Subscriber, Participant, Always)
  • Nikolay Shlyakhov(Reporter)