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
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: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.
Mistaken commas have been changed to semi-colons. Thanks!
Should "numLayerInOls" be "numLayersInOls" rather than never used variable?