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: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.)
comment:6 Changed 4 years ago by shanl
- Resolution set to fixed
- Status changed from reopened to closed
Mistaken commas have been changed to semi-colons. Thanks!
Should "numLayerInOls" be "numLayersInOls" rather than never used variable?