Opened 5 years ago
Closed 5 years ago
#773 closed defect (fixed)
Use of num_slices_in_tile_minus1[ i ] in Section 7.3.2.4
Reported by: | bheng | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | |
Component: | spec | Version: | VVC D7 vE |
Keywords: | Cc: | ksuehring, bbross, XiangLi, fbossen, jvet@… |
Description
The draft text version JVET-Q0041-v2 includes an edit that replaced:
numSlicesInTileMinus1 = num_slices_in_tile_minus1[ i ]
for( j = 0; j < numSlicesInTileMinus1; j++ )
slice_height_in_ctu_minus1[ i++ ]
with this:
for( j = 0; j < num_slices_in_tile_minus1[ i ]; j++ )
slice_height_in_ctu_minus1[ i++ ]
Given that "i" is incremented within the for loop, num_slices_in_tile_minus1[i] is not valid once "i" changes (inferred to be 0). So, I would recommend undoing this edit.
Change history (2)
comment:1 Changed 5 years ago by yk
comment:2 Changed 5 years ago by yk
- Resolution set to fixed
- Status changed from new to closed
Fixed by actions taken at the 17th JVET meeting.
Note: See TracTickets for help on using tickets.
Oh, crap, my mistake. Will revert that change next time when I take the token. Hmm, need to be more careful for changes of the loop counter inside the loop. Thank you!