Opened 4 years ago
Closed 4 years ago
#1196 closed defect (fixed)
Typo in variable names
Reported by: | bheng | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | |
Component: | spec | Version: | VVC D10 vA |
Keywords: | Cc: | ksuehring, bbross, XiangLi, fbossen, jvet@… |
Description
The position (xTbComp, yTbComp) is used in several places but never defined anywhere. I believe these should either be ( xTbCmp, yTbCmp ) or ( xCurr, yCurr ) depending on the context.
On a related note, several of the input names mentioned for 8.7.5.1 (just invoked as 8.7.5 in the text) don't match up with the names used in the actual 8.7.5.1 subclause.
( xCurr, yCurr ), nCurrSw, and nCurrSh
versus
( xTbComp, yTbComp ), nTbW, and nTbH
versus
( xB, yB ), bWidth, and bHeight
Change history (2)
comment:1 Changed 4 years ago by jlchen
comment:2 Changed 4 years ago by bbross
- Resolution set to fixed
- Status changed from new to closed
Good catch, thanks! Fixed the input parameters and the cross-reference to 8.7.5.1 (instead of the parent clause 8.7.5) in JVET-S2001-vE.
Thanks for reporting the bug. Those (xTbComp, yTbComp)(s) will be changed to ) (xTbCmp, yTbCmp)(s) in vE release
Regarding the input parameter names not matching up with the names inside the subclause, we will check whether this exist in many places or only a few places. And decide whether need to fix them or not.