================ @@ -244,24 +246,25 @@ void WalkAST::VisitCallExpr(CallExpr *CE) { if (containsBadStrlcpyStrlcatPattern(CE)) { const Expr *DstArg = CE->getArg(0); const Expr *LenArg = CE->getArg(2); - PathDiagnosticLocation Loc = - PathDiagnosticLocation::createBegin(LenArg, BR.getSourceManager(), AC); + PathDiagnosticLocation Loc = PathDiagnosticLocation::createBegin( + LenArg, BR.getSourceManager(), AC); StringRef DstName = getPrintableName(DstArg); SmallString<256> S; llvm::raw_svector_ostream os(S); - os << "The third argument allows to potentially copy more bytes than it should. "; + os << "The third argument allows to potentially copy more bytes than it " + "should. "; os << "Replace with the value "; ---------------- pogo59 wrote:
clang-format can't do that sort of thing automatically. Breaking the too-long line is correct IMO. https://github.com/llvm/llvm-project/pull/82599 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits