[clang-tools-extra] [clang-tidy] Fix fragile test in `read-parameters-from-file` (PR #171033)

via cfe-commits cfe-commits at lists.llvm.org
Sun Dec 7 04:10:00 PST 2025


https://github.com/zeyi2 created https://github.com/llvm/llvm-project/pull/171033

[CommandLine.cpp](https://github.com/llvm/llvm-project/blob/fb0400fe1f1f9e83f3148db8ce2c72ab5bc6728e/llvm/lib/Support/CommandLine.cpp#L940) treats single quote as literal characters on Windows, so the argument is parsed as a check named ' -*,llvm-namespace-comment ', which matches no existing checks, so no checks are enabled via the command line.

Previously, the test passed because it fell back to the root `.clang-tidy` configuration which enables `llvm-*`.

>From 41f4887c8e43d626ebe7a712ecc4bb83119fd60f Mon Sep 17 00:00:00 2001
From: mtx <mitchell.xu2 at gmail.com>
Date: Sun, 7 Dec 2025 20:03:24 +0800
Subject: [PATCH] [clang-tidy] Fix fragile test in `read-parameters-from-file`

---
 .../test/clang-tidy/infrastructure/Inputs/param/parameters.txt  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/clang-tools-extra/test/clang-tidy/infrastructure/Inputs/param/parameters.txt b/clang-tools-extra/test/clang-tidy/infrastructure/Inputs/param/parameters.txt
index a6d8fa7ee299f..5cf5693664f53 100644
--- a/clang-tools-extra/test/clang-tidy/infrastructure/Inputs/param/parameters.txt
+++ b/clang-tools-extra/test/clang-tidy/infrastructure/Inputs/param/parameters.txt
@@ -1,2 +1,2 @@
--checks='-*,llvm-namespace-comment'
+-checks=-*,llvm-namespace-comment
 --warnings-as-errors=llvm-namespace-comment



More information about the cfe-commits mailing list