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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
