FWIW: I disagree with the religious "put braces around everything" I think sometimes for really small if's, it's ok to do a one-liner

Eventually, we'll get clang-tidy to do it automatically. Also I ran into a few case that are very confusing, like:

if (...) dosomething;
{
  LOCK(...);
  // ...
}

There are a few of these in the codebase, taking advantage of RAII to unlock the mutex. Very confusing.

Eventually, we'll get clang-tidy to do it automatically. Also I ran into a few case that are very confusing, like:

if (...) dosomething;
{
  LOCK(...);
  // ...
}

There are a few of these in the codebase, taking advantage of RAII to unlock the mutex. Very confusing.

Ouch. I agree the above is a trap. As are a lot of the ones you fixed which vastly improved readability (good work!).

But honestly don't you think something like:

if ( !fFlag ) vec += " ";

Or something equally trivial .. putting braces around it sort of makes the function longer and arguably doesn't add much in terms of readability as a result of that sacrifice (less fits on screen, etc).

Anyway this stuff isn't hugely important. Like I say -- it's your project and you get to set the standard.