Misleading definition of limited EGk binarization, leading to errors in its invocation

[jonathang]

Definition of limited EGk binarization (section 9.3.3.6) currently depends on a "riceParam" variable rather than k. The variable name "log2TransformRange" is also misleading because the process is now used by both transform skip residual coding and MVD coding. Editorial suggestions are to replace "riceParam" with "k", and "log2TransformRange" with "truncSuffixLen" instead.

9.3.3.11 and 9.3.3.12 invoke 9.3.3.6 with k set equal to cRiceParam + 1 when k is not used by the binarisation. They also invoke with cRiceParam as an input variable (it is not).

9.3.3.14 does not have errors, but invokes 9.3.3.6 by redundantly setting both k=1 and riceParam=1.

[jonathang]

Suggested fix (tracked changes)

[bbross]

Good catch, thanks!

This will be fixed in the first version of D10.