Fix renaming of variables

[NeoQuix]

Proposal

WhileLoopRenamer/ForLoopRenamer/variablenamegeneration all rename stuff, but not really together. Variablenamegeneration manages to skip renamed variables, but with very nasty tricks which do not always work.

Approach

• move WhileLoopRenamer from readability_based_redefinement into variable_name_generation and keep it
• rewrite ForLoopRenamer because it really has some problems (with empty assignments or ListComps)
• change how variable_name_generation renames variables (via AST replace method)
• change variable_name_generation so that WhileLoopRenamer/ForLoopRenamer run AFTER it ==> rename everything to hungarian/something else, and afterwards rename for/while variable names.

[NeoQuix]

/cib

[github-actions[bot]]

Branch issue-303-Fix_renaming_of_variables created!

[github-actions[bot]]

Could not create PR (Not Found)

[NeoQuix]

Also fixes some bugs with hungarian (force for loops) ((but then ForLoopRenamer fails independently, yay))

[NeoQuix]

Closed because of split into 2 PR