Opened 8 years ago

Closed 8 years ago

#1272 closed defect (fixed)

Redundant condition in default weighted sample prediction process

Reported by: jackh Owned by:
Priority: minor Milestone: RExt D7
Component: RExt text Version: RExt D6 (P1005) v1
Keywords: Cc: davidf, joel, teruhiko, jct-vc@…

Description

In section 8.5.3.3.4.2 of P1005 v2, the following section appears:

– The variable shift1 is set equal to Max( 2, 14 − bitDepth ) and the variable shift2 is set equal to Max( 3, 15 − bitDepth ).
– The variable offset1 is derived as follows:
– If shift1 is greater than 0, offset1 is set equal to 1 << ( shift1 − 1 ).
– Otherwise (shift1 is equal to 0), offset1 is set equal to 0.

The condition on shift1 seems redundant, as shift1 can never be lower than 2.

Change History (2)

comment:1 Changed 8 years ago by DefaultCC Plugin

  • Cc davidf joel teruhiko jct-vc@… added

comment:2 Changed 8 years ago by davidf

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

Fixed in Q1005_v1

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

  • David Flynn(Subscriber, Participant)
  • Jack Haughton(Reporter)
  • jct-vc@…(Subscriber)
  • Joel(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Always)
  • Teruhiko Suzuki(Subscriber)