[PATCH] D117416: [clang-format] Handle C variables with name that matches c++ access specifier

[MyDeveloperDay via Phabricator via cfe-commits]

MyDeveloperDay accepted this revision.
MyDeveloperDay added a comment.

new nits but basically I think this looks like it might handle this ok? LGTM

We need to run this across a large code base to check



================
Comment at: clang/lib/Format/UnwrappedLineFormatter.cpp:106
+      bool returnValue{false};
+      if (RootToken.isAccessSpecifier(Style.isCpp())) {
+        returnValue = true;
----------------
remove extraneous `{}` braces


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D117416/new/

https://reviews.llvm.org/D117416

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits