================ @@ -164,6 +164,33 @@ void UseRangesCheck::registerMatchers(MatchFinder *Finder) { static void removeFunctionArgs(DiagnosticBuilder &Diag, const CallExpr &Call, ArrayRef<unsigned> Indexes, const ASTContext &Ctx) { + auto GetCommaLoc = + [&](SourceLocation BeginLoc, + SourceLocation EndLoc) -> std::optional<CharSourceRange> { + auto Invalid = false; + StringRef SourceText = Lexer::getSourceText( + CharSourceRange::getCharRange({BeginLoc, EndLoc}), + Ctx.getSourceManager(), Ctx.getLangOpts(), &Invalid); + assert(!Invalid); ---------------- PiotrZSL wrote:
if it's invalid, better would be simply to return std::nullopt, instead of crashing or running into some problems later. https://github.com/llvm/llvm-project/pull/118568 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits