Opened 18 months ago

Last modified 18 months ago

#1448 new enhancement

offset_len_minus1 range

Reported by: kolya Owned by:
Priority: minor Milestone:
Component: Text Version:
Keywords: offset_len_minus1 Cc: bbross, wjhan, jct-vc@…

Description

The spec says that valid range for offset_len_minus1 is [0, 31]. At the same time, offset_len_minus1=0 means that there is a slice segment of 1 byte size (entry_point_offset_minus1 == 0). But this is not possible coz every slice segment should contain at least 9 bits according to the clause 9.3.2.6.

Thus the valid range for offset_len_minus1 should be described as [1, 31] and valid range for entry_point_offset_minus1 is [1, 232 - 1].

(on behalf of pavel.evsikov@…)

Change History (2)

comment:1 follow-up: Changed 18 months ago by kiranmisra

Note, this issue was explicitly discussed at the JCTVC L meeting.

comment:2 in reply to: ↑ 1 Changed 18 months ago by kolya

Replying to kiranmisra:

Note, this issue was explicitly discussed at the JCTVC L meeting.

Here is following quotation from meeting notes:

JCTVC-L0116 High-level parallelism clean-ups [T. Lee, B. Choi (Samsung)]
This document proposes some high-level syntax changes related to dependent slice segment and entry points signalling. In this proposal, the memorization process for context variables for dependent slice segment is proposed to be invoked no matter what the current slice segment type is, and entry point offsets are coded as the decreased value by 1.
Decision (Ed.): The first aspect is just pointing out an error in the text; the "memorization" is necessary to be performed for independent slice segments as well as dependent slice segments.
The second aspect proposes to apply the "_minus1" coding convention for entry_point_offset[ i ] syntax elements. If we don't adopt this, we would probably want to explicitly prohibit the value 0 anyway, which would be a bit strange to do.
Decision (Cleanup): Apply the "_minus1" coding convention for entry_point_offset[ i ] syntax elements.

But for now, even with "_minus1" added, entry_point_offset_minus1[ i ] cannot be lower than 1.

Note: See TracTickets for help on using tickets.

This list contains all users that will be notified about changes made to this ticket.

These roles will be notified: Reporter, Owner, Subscriber, Participant

  • Benjamin Bross(Subscriber)
  • David Flynn(Always)
  • jct-vc@…(Subscriber)
  • Karl Sharman(Always)
  • Karsten Suehring(Always)
  • Kiran Misra(Participant)
  • Nikolay Shlyakhov(Reporter, Participant)
  • Woo-Jin Han(Subscriber)