This is in addition to generating Makefiles or Ninja files. It can beused by various LLVM tooling like clang-tidy or YouCompleteMe

Ran: clang-tidy -p . -header-filter=".*" "-checks=-*,performance-inefficient-string-concatenation" -fix **/*.cppTo solve: warning: string concatenation results in allocation of unnecessary temporary strings; consider using 'operator+=' or 'string::append()' instead [performance-inefficient-string-concatenation]

floor(3) as used previously is the libc floor, which takes a doublealways, so the cast to float is superfluous. floorf(3) would take afloat argument, but instead of switching to that, use std::floor whichhas the proper overload for double/float, and as of C++11 also has onefor integral types, so use that directly.The clang-tidy fixes were massaged manually, as it wasn't quite assmart.

The fixes that added std::move (C++11 only) were manually revertedagain. This sprinkles a ton of const refs all over the place, obviatingthe need to copy-construct Position all the time.