Fix comment typos.
Initialise all members.

Issue found by cppcheck.
Pre instead of post increment a variable.

Issue found by cppcheck.
Replace a increment loop with std::advance.

The code was flagged by cppcheck, due to the post increment.
Pre instead of post increment a variable.

Issue found by cppcheck.
Pre instead of post increment a variable.

Issue found by cppcheck.
Pre instead of post increment a variable.

Issue found by cppcheck.
Use empty() instead of comparing size() with 0.

Issue found by cppcheck.
Use empty() instead of comparing size() with 0.

Issue found by cppcheck.
Use dtor + placement new in assignment operator.

The change was used to simplify the code. A member was not copied and
that was noticed by cppcheck.
Use a variable as const ref instead of const.

Issue found by cppcheck.
Limit variable scope.

Issue found by cppcheck.
Remove an unneeded helper variable.

There was a cppcheck warning, that the scope of has_end_turn could be
reduced. Removing the variable makes the even simpler.
Limit variable scope.

Issue found by cppcheck.
Merge branch 'cppcheck'

Fixes several issues found by cppcheck, or issues found while reviewing
the issues found by cppcheck.