Having a consistent style actually helps a lot in large code bases. But these days, this can be automated with clang-format, so I agree that it's a bit silly that this is checked manually.

The bit about reusing i is a good catch though, that's not just a code-style thing but a real potential bug.