Opened 5 years ago

Closed 4 years ago

#306 closed enhancement (fixed)

Redundant clipping in 8.5.5.6

Reported by: chhuanb Owned by:
Priority: minor Milestone: VVC D5 v8
Component: spec Version: VVC D7 vE
Keywords: Cc: ksuehring, bbross, XiangLi, fbossen, jvet@…

Description

Formula (8-618) and (8-619) are redundant because cpMvLXConst6[ 0 ] is fetched from storage MVF.

cpMvLXConst6[ 0 ][ 0 ] = Clip3( −217, 217 − 1, cpMvLXConst6[ 0 ][ 0 ] ) (8-618)
cpMvLXConst6[ 0 ][ 1 ] = Clip3( −217, 217 − 1, cpMvLXConst6[ 0 ][ 1 ] ) (8-619)

Change history (11)

comment:1 Changed 5 years ago by bbross

  • Milestone set to VVC D5 v8

comment:2 Changed 5 years ago by bbross

  • Version changed from VVC D5 v7 to VVC D5 v8

comment:3 Changed 5 years ago by bbross

  • Version changed from VVC D5 v8 to VVC D5 v9

comment:4 Changed 5 years ago by bbross

  • Version changed from VVC D5 v9 to VVC D5 v10

comment:5 Changed 5 years ago by bbross

  • Version changed from VVC D5 v10 to VVC D6 vD

comment:6 Changed 5 years ago by bbross

  • Version changed from VVC D6 vD to VVC D6 vE

comment:7 Changed 5 years ago by bbross

  • Type changed from defect to enhancement

This seems to be purely editorial.
Would removing this redundancy make the text easier to read.

comment:8 Changed 4 years ago by bbross

  • Version changed from VVC D6 vE to VVC D7 vC

comment:9 Changed 4 years ago by bbross

  • Version changed from VVC D7 vC to VVC D7 vD

comment:10 Changed 4 years ago by bbross

  • Version changed from VVC D7 vD to VVC D7 vE

comment:11 Changed 4 years ago by jlchen

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

Good catch! To be fixed in the next release.

Note: See TracTickets for help on using tickets.