[clang] [clang-format] Fix a regression on BAS_AlwaysBreak (PR #107506)

Owen Pan via cfe-commits cfe-commits at lists.llvm.org
Sat Sep 7 16:49:24 PDT 2024


================
@@ -861,7 +861,7 @@ void ContinuationIndenter::addTokenOnCurrentLine(LineState &State, bool DryRun,
       //  or
       //  caaaaaaaaaaaaaaaaaaaaal(
       //       new SomethingElseeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee());
-      !IsSimpleFunction(Current)) {
+      Current.isNot(tok::comment) && !IsSimpleFunction(Current)) {
----------------
owenca wrote:

> would it make sense to introduce this into `IsSimpleFunction` ? similar to handling of `new` keyword in there?

I did that at first but thought it would be confusing to say that a comment is a simple function (or function call).

https://github.com/llvm/llvm-project/pull/107506


More information about the cfe-commits mailing list