Opened 4 years ago

Closed 4 years ago

#1232 closed defect (fixed)

Typos in VPS semantics

Reported by: bheng Owned by:
Priority: minor Milestone:
Component: spec Version: VVC D10 vD
Keywords: Cc: ksuehring, bbross, XiangLi, fbossen, jvet@…

Description

There are a couple for-loops where a comma is supposed to be a semi-colon.

for( i = 1, i <= vps_max_layers_minus1; i++ ) {

for( k = i − 1, k >= 0; k− − ) {

Also, is there a reason for the variable "numLayerInOls"? It's reset to 0, and then incremented in a couple places, but never used elsewhere as far as I can tell.

Change history (6)

comment:1 Changed 4 years ago by shanl

Mistaken commas have been changed to semi-colons. Thanks!

Should "numLayerInOls" be "numLayersInOls" rather than never used variable?

Last edited 4 years ago by shanl (previous) (diff)

comment:2 Changed 4 years ago by bbross

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

comment:3 Changed 4 years ago by yk

The variable, either in the form of "numLayerInOls" or "numLayersInOls", is not used anywhere in the spec. Thus its derivation has just been removed in a version being worked for JVET-S2001-vF.

comment:4 Changed 4 years ago by shanl

  • Resolution fixed deleted
  • Status changed from closed to reopened

comment:5 Changed 4 years ago by shanl

The variable "numLayersInOls" is used in multiple places in the spec.

"when NumLayersInOls[ i ] is greater than 0, are derived as follows..."

"...j is in the range of NumLayersInOls[ i ] − 1, ..."

"When NumLayersInOls[ i ] is equal to 1, the profile_tier_level( ) syntax structure..."

"When NumLayersInOls[ i ] is equal to 1, the general_timing_hrd_parameters( ) syntax..."

"If NumLayersInOls[ TargetOlsIdx ] is equal to 1, PicWidthMaxInSamplesY..."

and many more (a couple of dozen of places.)

Version 1, edited 4 years ago by shanl (previous) (next) (diff)

comment:6 Changed 4 years ago by shanl

  • Resolution set to fixed
  • Status changed from reopened to closed
Note: See TracTickets for help on using tickets.