Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#639 closed enhancement (fixed)

Redundant condition i < num_slices_in_pic_minus1 in PPS syntax

Reported by: vdrugeon Owned by:
Priority: minor Milestone:
Component: spec Version:
Keywords: Cc: ksuehring, bbross, XiangLi, fbossen, jvet@…

Description

The condition on presence of tile_idx_delta[ i ] is partially redundant: "if( tile_idx_delta_present_flag && i < num_slices_in_pic_minus1 )". Indeed, since this condition is within a for loop that finishes when i reaches num_slices_in_pic_minus1, the value of i will never be equal to num_slices_in_pic_minus1, so that "i < num_slices_in_pic_minus1" can be removed from the condition above.

Change history (3)

comment:1 Changed 4 years ago by yk

  • Resolution set to fixed
  • Status changed from new to closed

Good catch! Fixed in JVET-P2001-vC/v12.

comment:2 Changed 4 years ago by bheng

The value "i" is also incremented at the line:

slice_height_in_ctu_minus1[i++]

If the last tile in the picture is split into multiple slices, i will be equal to num_slices_in_pic_minus1 at the end of the picture. The condition mentioned here prevents the tile_idx_delta[i] from being sent in that case (since it is not necessary).

So, I would suggest undoing this change.

comment:3 Changed 4 years ago by yk

Aha, that's right, Tim! Undid.

Note: See TracTickets for help on using tickets.