[3.x] Style: Enforce braces around if blocks and loops #48475
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Continuation of #48447 and #48454 to modernize the
3.x
branch and fit the style used inmaster
.Note: This was done with
clang-tidy
(+ fixing its often bogus changes manually) and therefore it only affects the code which I was compiling (p=x11 tools=yes target=debug
). So there's still missing braces in other platform-specific code and within some preprocessor defines which were not set in my local build. If anyone wants to fix what was missed here, feel free to, but having a perfectly harmonious style is not a priority for3.x
, so such effort is best spent onmaster
.