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

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!

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.