Opened 4 years ago
Closed 4 years ago
#1314 closed defect (fixed)
Unnecessary variables in Section 8.5.2.10
Reported by: | bheng | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | |
Component: | spec | Version: | VVC D10 vE |
Keywords: | Cc: | ksuehring, bbross, XiangLi, fbossen, jvet@… |
Description
The following sentence seems unnecessary since currCB and currPic are not used elsewhere in this section:
The variable currCb specifies the current luma coding block at luma location ( xCb, yCb ) and the variable currPic specifies the current picture.
Equations 581 and 583 also seem unnecessary, since refIdxB is not an output of this subclause and is used elsewhere.
refIdxB = RefIdxLX[ xNbBk ][ yNbBk ] (581)
refIdxB = RefIdxLY[ xNbBk ][ yNbBk ] (583)
Change history (3)
comment:1 Changed 4 years ago by jlchen
- Resolution set to fixed
- Status changed from new to closed
comment:2 Changed 4 years ago by bheng
- Resolution fixed deleted
- Status changed from closed to reopened
Thanks for all the fixes!
It looks like this sentence is still in the draft:
"The variable currCb specifies the current luma coding block at luma location ( xCb, yCb ) and the variable currPic specifies the current picture."
Probably you could remove it unless you've noticed something I missed.
comment:3 Changed 4 years ago by bbross
- Resolution set to fixed
- Status changed from reopened to closed
Thanks for checking and reopening! The unnecessary sentence will be removed in JVET-S2001-vH.
Good catch, Brian! Those text look redundant to me. Will be fixed in S2001-vH release.