[PATCH] D60320: [clang-format]: Add option to insert space after locical not operator

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Apr 7 05:31:13 PDT 2019


MyDeveloperDay accepted this revision.
MyDeveloperDay added a comment.
This revision is now accepted and ready to land.

LGTM (minor nit)



================
Comment at: clang/lib/Format/TokenAnnotator.cpp:2834
   }
-  if (Left.is(TT_UnaryOperator))
-    return Right.is(TT_BinaryOperator);
+  if (Left.is(TT_UnaryOperator)) {
+    return (Style.SpaceAfterLogicalNot && Left.is(tok::exclaim)) ||
----------------
nit: you don't need to {} 


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

https://reviews.llvm.org/D60320





More information about the cfe-commits mailing list