search

klenin / Spawner

Cross-platform sandbox runner
4 stars 8 forks source link

Use C++11 range-based for-loop syntax where possible. #83

Closed cher-nov closed 7 years ago

cher-nov commented 7 years ago

This also fixes unnecessary usage of references for primitive types like pointers.

cher-nov commented 7 years ago

Conversely, when changing from index variables (appropriately named i) to elements it would be good to change the name, but you did not.

I somehow decided that you will not approve such changes, so I just unified the minority of names with the majority. Ok, I was wrong.

Separated all name changes to another PR. I will propose it if this will be merged. Also split this PR to smaller commits. Please review.