[PATCH] D156259: Fix a bug that erroneously placed function arguments on a new line despite all arguments being able to fit on the same line.

Thu Jul 27 09:13:50 PDT 2023

jp4a50 added inline comments.

================
Comment at: clang/lib/Format/ContinuationIndenter.cpp:664-666
+    if ((Style.Language == FormatStyle::LK_Cpp ||
+         Style.Language == FormatStyle::LK_ObjC) &&
+        !Current.is(tok::comment) && PrevNonComment &&
----------------
HazardyKnusperkeks wrote:
> jp4a50 wrote:
> > HazardyKnusperkeks wrote:
> > > Could move this out of the lambda.
> > Not sure exactly what you're referring to here. Initialization of `PrevNonComment`?
> > Not sure exactly what you're referring to here. Initialization of `PrevNonComment`?
> 
> No everything in the if before you go into the `PrevNonComment`. It is highlighted if you hover the comment.
Ah, I see, thanks. Technically I could move that outside the lambda. I'm just not sure how it's better? Are you making the suggestion because it would make it clearer that this logic only applies to cpp/objc and/or because it means we could avoid capturing `Style`?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156259