This revision was automatically updated to reflect the committed changes.
Closed by commit rGba12cec21f55: [clang-format] [PR43531] clang-format damages
alternative representations for… (authored by MyDeveloperDay).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
mitchell-stellar accepted this revision.
mitchell-stellar added a comment.
This revision is now accepted and ready to land.
LGTM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D68332/new/
https://reviews.llvm.org/D68332
___
cfe-commits
MyDeveloperDay updated this revision to Diff 223215.
MyDeveloperDay marked 3 inline comments as done.
MyDeveloperDay added a comment.
- add additional compl(5) and not(5) unit tests
- improve the comments around the condition and explaining the tests
- remove the use of .equals(...)
CHANGES
mitchell-stellar requested changes to this revision.
mitchell-stellar added inline comments.
This revision now requires changes to proceed.
Comment at: clang/lib/Format/TokenAnnotator.cpp:2892
+ if (Left.is(TT_UnaryOperator)) {
+// Don't combine the unary operators !~ into
MyDeveloperDay created this revision.
MyDeveloperDay added reviewers: klimek, reuk, owenpan, mitchell-stellar,
STL_MSFT.
MyDeveloperDay added projects: clang-format, clang-tools-extra.
Herald added a project: clang.
https://bugs.llvm.org/show_bug.cgi?id=43531
Fix for clang-format incorrectly
