[PATCH] D75621: [clang-tidy] Use ; as separator for HeaderFileExtensions

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 5 22:19:44 PST 2020


lebedev.ri added a comment.

FWIW i'm personally still unconvinced this is fixing a real problem



================
Comment at: clang-tools-extra/clang-tidy/utils/FileExtensionsUtils.cpp:43-45
+        llvm::errs()
+            << "Using ',' as a file extension delimiter is deprecated. Please "
+               "switch your configuration to use ';'.\n";
----------------
Can this spam be avoided?
It's going to be really intrusive, and it is not always possible to
just perform the migration (think: using more than one clang-tidy version.)
The deprecation message should be in the docs/releasenotes.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D75621/new/

https://reviews.llvm.org/D75621





More information about the cfe-commits mailing list