Calling of arguments in clause 8.5.2.14

[chujoh]

There are two mistakes of calling for arguments in clause 8.5.2.14, which specifies the rounding process for motion vectors.

8.5.2.4 Derivation process for pairwise average merging candidate

– The rounding process for motion vectors as specified in clause 8.5.2.14 is invoked with mvX[ 0 ] set equal to mvLXp0Cand[ 0 ] + mvLXp1Cand[ 0 ], mvX[ 1 ] set equal to mvLXp0Cand[ 1 ] + mvLXp1Cand[ 1 ], rightShift set equal to1, and leftShift set equal to 0 as inputs and the rounded mvLXavgCand[ 0 ] as output.

The above sentence should be improved by the following.

– The rounding process for motion vectors as specified in clause 8.5.2.14 is invoked with mvX[ 0 ] set equal to mvLXp0Cand[ 0 ] + mvLXp1Cand[ 0 ], mvX[ 1 ] set equal to mvLXp0Cand[ 1 ] + mvLXp1Cand[ 1 ], rightShift set equal to1, and leftShift set equal to 0 as inputs and the rounded mvLXavgCand as output.

8.5.5.9 Derivation process for motion vector arrays from affine control point motion vectors
– For i = 0..1, the following applies:
– The rounding process for motion vectors as specified in clause 8.5.2.14 is invoked with mvX set equal to diffMvLX[ x ][ y ][ i ], rightShift set equal to 8, and leftShift set equal to 0 as inputs and the rounded diffMvLX[ x ][ y ][ i ] as output.
– The value of diffMvLX[ x ][ y ][ i ] is clipped as follows:
diffMvLX[ x ][ y ][ i ] = Clip3( −dmvLimit + 1, dmvLimit − 1, diffMvLX[ x ][ y ][ i ]) (918)

The above paragraph should be improved by the following.

– The rounding process for motion vectors as specified in clause 8.5.2.14 is invoked with mvX set equal to diffMvLX[ x ][ y ], rightShift set equal to 8, and leftShift set equal to 0 as inputs and the rounded diffMvLX[ x ][ y ] as output.
– For i = 0..1, the following applies:
– The value of diffMvLX[ x ][ y ][ i ] is clipped as follows:
diffMvLX[ x ][ y ][ i ] = Clip3( −dmvLimit + 1, dmvLimit − 1, diffMvLX[ x ][ y ][ i ]) (918)

[bbross]

Good catch, thanks!

This will be fixed in first verison of D10.