[clang-tools-extra] [clang-tidy][clangd] Fixed removeFunctionArgs don't remove comma for use-ranges check (PR #118568)

Piotr Zegar via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 13 12:00:49 PST 2025


================
@@ -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> {
----------------
PiotrZSL wrote:

you could actually use "invalid source range" instead of optional

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


More information about the cfe-commits mailing list