[clang] Extend -Wnonportable-include-path with a warning about trailing whitespace or dots (PR #96960)

Tobias Mayer via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 28 08:30:30 PDT 2024


================
@@ -361,7 +361,11 @@ class NonportablePath  : Warning<
   "non-portable path to file '%0'; specified path differs in case from file"
   " name on disk">;
 def pp_nonportable_path : NonportablePath,
-  InGroup<DiagGroup<"nonportable-include-path">>;
+  InGroup<NonportablePath>;
+def pp_nonportable_path_trailing_whitespace : Warning<
+  "non-portable path to file '%0'; specified path contains trailing"
+  "whitespace or dots">,
----------------
tobias-mayer wrote:

Sounds good. How should we handle a case like ` #include "test.h.. .. "` where we have both? Should we add a third option complaining about both or just pick whatever is the last character?

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


More information about the cfe-commits mailing list