In my experience it has these advantages:
- harder type checks,
- allows to use = default ctor,
- simplify ctors.
// I hope this time, I've created "ok" diff.
Did you make these changes manually, or use clang-tidy for it?
Personally, I would prefer the = member init syntax over the braced initialization. This is how we did it in most other cases already as well where we use in-class member initialization.