Resolve "Non matching variable found in function unclr_calc_corr_features_fx()"

Vladimir Malenovskyrequested to merge
basop-2579-non-matching-variable-found-in-function-unclr_calc_corr_features_fx into main

Reason why this change is needed

Static code analysis shows there is a missmatch between a certain variable across float and fixed point versions.

In floating point implementation we have:

corrLagMax = maximum( corrEst, ( lagSearchRange[1] - lagSearchRange[0] + 1 ), &corrEstMax );

but in fixed point there is:

corrLagMax = maximum_l( corrEst, sub( lagSearchRange[1], add( lagSearchRange[0], 1 ) ), &corrEstMax );

There is a difference of 2 between these versions that can cause bugs.

Please fix it or provide proper justification for it.

Description of the change

The order of add and sub operations is corrected.

Affected operating points

  • Non-BE change affect optentially several op. points invovling STEREO core coder

Merge request reports