Tidy ups to variable and argument names

[jhol]

This patch-set makes all variables and arguments consistent with lower_case snake_case style, with a underscore prefix applied to local variables.

[valgur]

The _ variable prefixes are unnecessary in function-s, since their scope is limited only to the function. I would keep them only for variables created inside macro-s, which do have a global scope.

[jhol]

The _ variable prefixes are unnecessary in function-s, since their scope is limited only to the function. I would keep them only for variables created inside macro-s, which do have a global scope.

Ok, no _ variable prefixes in functions now, except where they are used as a shadows of output variables.

[jhol]

Can we have this merged?

[memsharded]

Hi @jhol

The CI is red, this cannot be merged until it is green, could you please have a look?

[jhol]

Hi @jhol

The CI is red, this cannot be merged until it is green, could you please have a look?

My mistake. I thought the test system was non-functional for this branch. The build issues should be fixed now.

[jhol]

This should be ready for merge now. Is there anything I can do to help get this accepted?

[memsharded]

Looks good to me, thanks for the contribution.