Local variable cleanup

First batch of formatting fixes, submitting these before things get massively out of sync again.

I've tried to apply the following rules throughout the code:

• Local variables should be declared within the scope in which they are used.
• Read-only variables should be declared const.
• Local variables used in only a limit range of the scope should get their own scope.

I've also tried being a bit more explicit in variable and function naming.

mentioned in merge request !114 (merged)

Shouldn't N be a size_t as well here ?

Added 21 commits:

• 0eb6bf61...1c1d5bdd - 20 commits from branch master
• cc4c84ba - Merge branch 'master' into local_variable_cleanup

The merge request needed updating against master, so I've done that. Better give it a once over before I accept.

This should be fixed in the !114 (merged) merge request.

LGTM!

mentioned in commit 1f9e985c

Added 11 commits:

• cc4c84ba...1f9e985c - 10 commits from branch master
• 1b316fc5 - Merge branch 'master' into local_variable_cleanup

Another merge conflict with size_t, fixed that and accepting quickly..

OK to remove branch?

mentioned in commit 3e0e0c69

Status changed to merged

Please don't remove the branch! Will clean up the for-loops next :)

OK, remember to pull this branch before making any further changes.

Will do, thanks!