Error with brackets and operator precedence in equation 433

[peterderivaz]

Equation 433 reads:

tmpVal = ( PaletteEscapeVal[ cIdx ][ xCbL + xL ][ yCbL + yL ] * levelScale[ qP%6 ] ) << ( qP / 6 ) + 32 ) >> 6

There are two opening brackets (, and three closing brackets ).

In addition, removing a closing bracket or placing an extra open bracket in any position seems incorrect because + has higher precedence than << according to table 1.

It looks like this might correspond to the following code from DecCu.cpp:544

value = ((((escapeValue.at(posXC, posYC)*g_invQuantScales[0][qpRem]) << qpPer) + add) >> invquantiserRightShift);

suggesting that the correct equation might be:

tmpVal = ( ( ( PaletteEscapeVal[ cIdx ][ xCbL + xL ][ yCbL + yL ] * levelScale[ qP%6 ] ) << ( qP / 6 ) ) + 32 ) >> 6

[bbross]

Good catch, thanks!
Your suggestion is correct and this will be fixed in v2 of JVET-W1004.