[clang-tools-extra] [clangd] Allow specifying what headers are always included via "" or <> (PR #67749)
kleines Filmröllchen via cfe-commits
cfe-commits at lists.llvm.org
Thu Jan 25 06:16:29 PST 2024
================
@@ -296,6 +296,19 @@ struct Fragment {
// ::). All nested namespaces are affected as well.
// Affects availability of the AddUsing tweak.
std::vector<Located<std::string>> FullyQualifiedNamespaces;
+
+ /// List of regexes for headers that should always be included with a
+ /// ""-style include. By default, and in case of a conflict with
+ /// AngledHeaders (i.e. a header matches a regex in both QuotedHeaders and
+ /// AngledHeaders), system headers use <> and non-system headers use "".
+ /// These can match any suffix of the header file in question.
----------------
kleinesfilmroellchen wrote:
We're matching against the spelling; at least in SerenityOS this is a necessary distinction: Naming a same-directory header via `"Widget.h"` will result in quotes, but naming the same header via an "absolute" path like `<LibGUI/Widget.h>` should get you an angled include.
https://github.com/llvm/llvm-project/pull/67749
More information about the cfe-commits
mailing list