[clang-tidy] Merge ClangTidyModuleRegistry.h into ClangTidyModule.h (#173231)
As far as I can tell, the only effect of having `ClangTidyModuleRegistry.h` be a separate header is that it forces you to include two headers instead of one before you can actually define your clang-tidy module.
This commit is contained in:
Victor Chernyakin
GitHub
parent
57683c6bb4
commit
1bcf74006b
@ -17,7 +17,7 @@
|
||||
#include "ClangTidy.h"
|
||||
#include "ClangTidyCheck.h"
|
||||
#include "ClangTidyDiagnosticConsumer.h"
|
||||
#include "ClangTidyModuleRegistry.h"
|
||||
#include "ClangTidyModule.h"
|
||||
#include "ClangTidyProfiling.h"
|
||||
#include "ExpandModularHeadersPPCallbacks.h"
|
||||
#include "clang-tidy-config.h"
|
||||
|
||||
@ -12,6 +12,7 @@
|
||||
#include "ClangTidyOptions.h"
|
||||
#include "llvm/ADT/StringMap.h"
|
||||
#include "llvm/ADT/StringRef.h"
|
||||
#include "llvm/Support/Registry.h"
|
||||
#include <functional>
|
||||
#include <memory>
|
||||
|
||||
@ -95,6 +96,12 @@ public:
|
||||
virtual ClangTidyOptions getModuleOptions();
|
||||
};
|
||||
|
||||
using ClangTidyModuleRegistry = llvm::Registry<ClangTidyModule>;
|
||||
|
||||
} // namespace clang::tidy
|
||||
|
||||
namespace llvm {
|
||||
extern template class Registry<clang::tidy::ClangTidyModule>;
|
||||
} // namespace llvm
|
||||
|
||||
#endif // LLVM_CLANG_TOOLS_EXTRA_CLANG_TIDY_CLANGTIDYMODULE_H
|
||||
|
||||
@ -9,17 +9,13 @@
|
||||
#ifndef LLVM_CLANG_TOOLS_EXTRA_CLANG_TIDY_CLANGTIDYMODULEREGISTRY_H
|
||||
#define LLVM_CLANG_TOOLS_EXTRA_CLANG_TIDY_CLANGTIDYMODULEREGISTRY_H
|
||||
|
||||
// NOLINTBEGIN
|
||||
|
||||
// TODO(LLVM 24) Delete this header.
|
||||
#warning The ClangTidyModuleRegistry.h header is deprecated and will be removed in LLVM 24. All of the symbols it used to define have been moved into ClangTidyModule.h.
|
||||
|
||||
#include "ClangTidyModule.h"
|
||||
#include "llvm/Support/Registry.h"
|
||||
|
||||
namespace clang::tidy {
|
||||
|
||||
using ClangTidyModuleRegistry = llvm::Registry<ClangTidyModule>;
|
||||
|
||||
} // namespace clang::tidy
|
||||
|
||||
namespace llvm {
|
||||
extern template class Registry<clang::tidy::ClangTidyModule>;
|
||||
} // namespace llvm
|
||||
// NOLINTEND
|
||||
|
||||
#endif // LLVM_CLANG_TOOLS_EXTRA_CLANG_TIDY_CLANGTIDYMODULEREGISTRY_H
|
||||
|
||||
@ -7,7 +7,7 @@
|
||||
//===----------------------------------------------------------------------===//
|
||||
|
||||
#include "ClangTidyOptions.h"
|
||||
#include "ClangTidyModuleRegistry.h"
|
||||
#include "ClangTidyModule.h"
|
||||
#include "clang/Basic/DiagnosticIDs.h"
|
||||
#include "clang/Basic/LLVM.h"
|
||||
#include "llvm/ADT/SmallString.h"
|
||||
|
||||
@ -8,7 +8,6 @@
|
||||
|
||||
#include "../ClangTidy.h"
|
||||
#include "../ClangTidyModule.h"
|
||||
#include "../ClangTidyModuleRegistry.h"
|
||||
#include "CleanupCtadCheck.h"
|
||||
#include "DurationAdditionCheck.h"
|
||||
#include "DurationComparisonCheck.h"
|
||||
|
||||
@ -8,7 +8,6 @@
|
||||
|
||||
#include "../ClangTidy.h"
|
||||
#include "../ClangTidyModule.h"
|
||||
#include "../ClangTidyModuleRegistry.h"
|
||||
#include "IdDependentBackwardBranchCheck.h"
|
||||
#include "KernelNameRestrictionCheck.h"
|
||||
#include "SingleWorkItemBarrierCheck.h"
|
||||
|
||||
@ -8,7 +8,6 @@
|
||||
|
||||
#include "../ClangTidy.h"
|
||||
#include "../ClangTidyModule.h"
|
||||
#include "../ClangTidyModuleRegistry.h"
|
||||
#include "CloexecAccept4Check.h"
|
||||
#include "CloexecAcceptCheck.h"
|
||||
#include "CloexecCreatCheck.h"
|
||||
|
||||
@ -8,7 +8,6 @@
|
||||
|
||||
#include "../ClangTidy.h"
|
||||
#include "../ClangTidyModule.h"
|
||||
#include "../ClangTidyModuleRegistry.h"
|
||||
#include "UseRangesCheck.h"
|
||||
#include "UseToStringCheck.h"
|
||||
using namespace clang::ast_matchers;
|
||||
|
||||
@ -8,7 +8,6 @@
|
||||
|
||||
#include "../ClangTidy.h"
|
||||
#include "../ClangTidyModule.h"
|
||||
#include "../ClangTidyModuleRegistry.h"
|
||||
#include "ArgumentCommentCheck.h"
|
||||
#include "AssertSideEffectCheck.h"
|
||||
#include "AssignmentInIfConditionCheck.h"
|
||||
|
||||
@ -8,7 +8,6 @@
|
||||
|
||||
#include "../ClangTidy.h"
|
||||
#include "../ClangTidyModule.h"
|
||||
#include "../ClangTidyModuleRegistry.h"
|
||||
#include "../bugprone/BadSignalToKillThreadCheck.h"
|
||||
#include "../bugprone/CommandProcessorCheck.h"
|
||||
#include "../bugprone/CopyConstructorMutatesArgumentCheck.h"
|
||||
|
||||
@ -8,7 +8,6 @@
|
||||
|
||||
#include "../ClangTidy.h"
|
||||
#include "../ClangTidyModule.h"
|
||||
#include "../ClangTidyModuleRegistry.h"
|
||||
#include "MtUnsafeCheck.h"
|
||||
#include "ThreadCanceltypeAsynchronousCheck.h"
|
||||
|
||||
|
||||
@ -8,7 +8,6 @@
|
||||
|
||||
#include "../ClangTidy.h"
|
||||
#include "../ClangTidyModule.h"
|
||||
#include "../ClangTidyModuleRegistry.h"
|
||||
#include "../bugprone/NarrowingConversionsCheck.h"
|
||||
#include "../misc/NonPrivateMemberVariablesInClassesCheck.h"
|
||||
#include "../misc/UnconventionalAssignOperatorCheck.h"
|
||||
|
||||
@ -1,6 +1,5 @@
|
||||
#include "../ClangTidy.h"
|
||||
#include "../ClangTidyModule.h"
|
||||
#include "../ClangTidyModuleRegistry.h"
|
||||
#include "../ClangTidyOptions.h"
|
||||
#include "QueryCheck.h"
|
||||
#include "llvm/ADT/SmallSet.h"
|
||||
|
||||
@ -8,7 +8,6 @@
|
||||
|
||||
#include "../ClangTidy.h"
|
||||
#include "../ClangTidyModule.h"
|
||||
#include "../ClangTidyModuleRegistry.h"
|
||||
#include "AvoidSpinlockCheck.h"
|
||||
#include "DispatchOnceNonstaticCheck.h"
|
||||
|
||||
|
||||
@ -8,7 +8,6 @@
|
||||
|
||||
#include "../ClangTidy.h"
|
||||
#include "../ClangTidyModule.h"
|
||||
#include "../ClangTidyModuleRegistry.h"
|
||||
#include "../google/UnnamedNamespaceInHeaderCheck.h"
|
||||
#include "../misc/MultipleInheritanceCheck.h"
|
||||
#include "DefaultArgumentsCallsCheck.h"
|
||||
|
||||
@ -8,7 +8,6 @@
|
||||
|
||||
#include "../ClangTidy.h"
|
||||
#include "../ClangTidyModule.h"
|
||||
#include "../ClangTidyModuleRegistry.h"
|
||||
#include "../modernize/AvoidCStyleCastCheck.h"
|
||||
#include "../readability/BracesAroundStatementsCheck.h"
|
||||
#include "../readability/FunctionSizeCheck.h"
|
||||
|
||||
@ -8,7 +8,6 @@
|
||||
|
||||
#include "../ClangTidy.h"
|
||||
#include "../ClangTidyModule.h"
|
||||
#include "../ClangTidyModuleRegistry.h"
|
||||
#include "../bugprone/UndelegatedConstructorCheck.h"
|
||||
#include "../bugprone/UseAfterMoveCheck.h"
|
||||
#include "../cppcoreguidelines/AvoidGotoCheck.h"
|
||||
|
||||
@ -8,7 +8,6 @@
|
||||
|
||||
#include "../ClangTidy.h"
|
||||
#include "../ClangTidyModule.h"
|
||||
#include "../ClangTidyModuleRegistry.h"
|
||||
#include "MustCheckErrsCheck.h"
|
||||
|
||||
namespace clang::tidy {
|
||||
|
||||
@ -8,7 +8,6 @@
|
||||
|
||||
#include "../ClangTidy.h"
|
||||
#include "../ClangTidyModule.h"
|
||||
#include "../ClangTidyModuleRegistry.h"
|
||||
#include "../readability/ElseAfterReturnCheck.h"
|
||||
#include "../readability/NamespaceCommentCheck.h"
|
||||
#include "../readability/QualifiedAutoCheck.h"
|
||||
|
||||
@ -8,7 +8,6 @@
|
||||
|
||||
#include "../ClangTidy.h"
|
||||
#include "../ClangTidyModule.h"
|
||||
#include "../ClangTidyModuleRegistry.h"
|
||||
#include "CalleeNamespaceCheck.h"
|
||||
#include "ImplementationInNamespaceCheck.h"
|
||||
#include "InlineFunctionDeclCheck.h"
|
||||
|
||||
@ -8,7 +8,6 @@
|
||||
|
||||
#include "../ClangTidy.h"
|
||||
#include "../ClangTidyModule.h"
|
||||
#include "../ClangTidyModuleRegistry.h"
|
||||
#include "ConfusableIdentifierCheck.h"
|
||||
#include "ConstCorrectnessCheck.h"
|
||||
#include "CoroutineHostileRAIICheck.h"
|
||||
|
||||
@ -8,7 +8,6 @@
|
||||
|
||||
#include "../ClangTidy.h"
|
||||
#include "../ClangTidyModule.h"
|
||||
#include "../ClangTidyModuleRegistry.h"
|
||||
#include "AvoidBindCheck.h"
|
||||
#include "AvoidCArraysCheck.h"
|
||||
#include "AvoidCStyleCastCheck.h"
|
||||
|
||||
@ -8,7 +8,6 @@
|
||||
|
||||
#include "../ClangTidy.h"
|
||||
#include "../ClangTidyModule.h"
|
||||
#include "../ClangTidyModuleRegistry.h"
|
||||
#include "BufferDerefCheck.h"
|
||||
#include "TypeMismatchCheck.h"
|
||||
|
||||
|
||||
@ -8,7 +8,6 @@
|
||||
|
||||
#include "../ClangTidy.h"
|
||||
#include "../ClangTidyModule.h"
|
||||
#include "../ClangTidyModuleRegistry.h"
|
||||
#include "AssertEqualsCheck.h"
|
||||
#include "AvoidNSErrorInitCheck.h"
|
||||
#include "DeallocInCategoryCheck.h"
|
||||
|
||||
@ -8,7 +8,6 @@
|
||||
|
||||
#include "../ClangTidy.h"
|
||||
#include "../ClangTidyModule.h"
|
||||
#include "../ClangTidyModuleRegistry.h"
|
||||
#include "ExceptionEscapeCheck.h"
|
||||
#include "UseDefaultNoneCheck.h"
|
||||
|
||||
|
||||
@ -8,7 +8,6 @@
|
||||
|
||||
#include "../ClangTidy.h"
|
||||
#include "../ClangTidyModule.h"
|
||||
#include "../ClangTidyModuleRegistry.h"
|
||||
#include "AvoidEndlCheck.h"
|
||||
#include "EnumSizeCheck.h"
|
||||
#include "FasterStringFindCheck.h"
|
||||
|
||||
@ -8,7 +8,6 @@
|
||||
|
||||
#include "../ClangTidy.h"
|
||||
#include "../ClangTidyModule.h"
|
||||
#include "../ClangTidyModuleRegistry.h"
|
||||
#include "AvoidPragmaOnceCheck.h"
|
||||
#include "RestrictSystemIncludesCheck.h"
|
||||
#include "SIMDIntrinsicsCheck.h"
|
||||
|
||||
@ -8,7 +8,6 @@
|
||||
|
||||
#include "../ClangTidy.h"
|
||||
#include "../ClangTidyModule.h"
|
||||
#include "../ClangTidyModuleRegistry.h"
|
||||
#include "AmbiguousSmartptrResetCallCheck.h"
|
||||
#include "AvoidConstParamsInDeclsCheck.h"
|
||||
#include "AvoidNestedConditionalOperatorCheck.h"
|
||||
|
||||
@ -8,7 +8,6 @@
|
||||
|
||||
#include "../ClangTidy.h"
|
||||
#include "../ClangTidyModule.h"
|
||||
#include "../ClangTidyModuleRegistry.h"
|
||||
#include "../fuchsia/TemporaryObjectsCheck.h"
|
||||
|
||||
namespace clang::tidy {
|
||||
|
||||
@ -10,7 +10,6 @@
|
||||
#include "../clang-tidy/ClangTidyCheck.h"
|
||||
#include "../clang-tidy/ClangTidyDiagnosticConsumer.h"
|
||||
#include "../clang-tidy/ClangTidyModule.h"
|
||||
#include "../clang-tidy/ClangTidyModuleRegistry.h"
|
||||
#include "../clang-tidy/ClangTidyOptions.h"
|
||||
#include "AST.h"
|
||||
#include "CollectMacros.h"
|
||||
|
||||
@ -7,7 +7,7 @@
|
||||
//===----------------------------------------------------------------------===//
|
||||
|
||||
#include "TidyProvider.h"
|
||||
#include "../clang-tidy/ClangTidyModuleRegistry.h"
|
||||
#include "../clang-tidy/ClangTidyModule.h"
|
||||
#include "../clang-tidy/ClangTidyOptions.h"
|
||||
#include "Config.h"
|
||||
#include "support/FileCache.h"
|
||||
|
||||
@ -25,7 +25,6 @@
|
||||
//===----------------------------------------------------------------------===//
|
||||
|
||||
#include "../clang-tidy/ClangTidyModule.h"
|
||||
#include "../clang-tidy/ClangTidyModuleRegistry.h"
|
||||
#include "../clang-tidy/ClangTidyOptions.h"
|
||||
#include "../clang-tidy/GlobList.h"
|
||||
#include "ClangdLSPServer.h"
|
||||
|
||||
@ -13,7 +13,6 @@
|
||||
|
||||
#include "../../clang-tidy/ClangTidyCheck.h"
|
||||
#include "../../clang-tidy/ClangTidyModule.h"
|
||||
#include "../../clang-tidy/ClangTidyModuleRegistry.h"
|
||||
#include "AST.h"
|
||||
#include "Config.h"
|
||||
#include "Diagnostics.h"
|
||||
|
||||
@ -86,6 +86,10 @@ Potentially Breaking Changes
|
||||
`-header-filter='.*'`. To disable warnings from non-system, set `-header-filter`
|
||||
to an empty string.
|
||||
|
||||
- The ``clang-tidy/ClangTidyModuleRegistry.h`` header has been deprecated and will
|
||||
be removed in LLVM 24. All of the symbols it used to define have been moved into
|
||||
``clang-tidy/ClangTidyModule.h``.
|
||||
|
||||
Improvements to clangd
|
||||
----------------------
|
||||
|
||||
|
||||
@ -96,7 +96,6 @@ The Directory Structure
|
||||
|-- ClangTidy.h # Interfaces for users.
|
||||
|-- ClangTidyCheck.h # Interfaces for checks.
|
||||
|-- ClangTidyModule.h # Interface for clang-tidy modules.
|
||||
|-- ClangTidyModuleRegistry.h # Interface for registering of modules.
|
||||
...
|
||||
|-- google/ # Google clang-tidy module.
|
||||
|-+
|
||||
|
||||
@ -10,7 +10,6 @@
|
||||
#include "clang-tidy/ClangTidy.h"
|
||||
#include "clang-tidy/ClangTidyCheck.h"
|
||||
#include "clang-tidy/ClangTidyModule.h"
|
||||
#include "clang-tidy/ClangTidyModuleRegistry.h"
|
||||
#include "clang/AST/ASTContext.h"
|
||||
#include "clang/ASTMatchers/ASTMatchFinder.h"
|
||||
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user