Declare local variables at top of scope.

Add space between ( and ?

Remove blank line.

Why redeclare result?

Remove blank line.

Why are you redeclaring result? This assignment will not survive outside of the if statement.

Remove blank line.

Remove blank line.

instead of 0, we can use the level Enum 'LVL_STATE_EMPTY'?

lower? and change the D98 level sensor comment as well.

Remove empty line

Space before LVL_STATE_LOW

space inside array index.

Space inside parenthesis.

Change this to LEVEL_T enum.

Is this macro still being used? if not, delete it.

This is total mess up . that is what I want it a clean implementation. created a ticket LDT-3563 and closing all review comments.

Removing all the review comments and created a ticket to handle it in separate ticket -LDT-3563

removed.

changed.

removed.

the 2nd if statement for beta 1.0 is an ask from systems team to raise fault alarm if user by mistake enable Beta 2.0 config when Beta1.0 is active