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

Last edited 4 years ago by shanl (previous) (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.