maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity
llvm-project/clang-tools-extra/clang-tidy/misc/MiscTidyModule.cpp

54 lines
1.9 KiB
C++
Raw Normal View History

Added a module for checks not related to LLVM or Google coding style. llvm-svn: 202970
2014-03-05 21:14:32 +08:00
//===--- MiscTidyModule.cpp - clang-tidy ----------------------------------===//
//
// The LLVM Compiler Infrastructure
//
// This file is distributed under the University of Illinois Open Source
// License. See LICENSE.TXT for details.
//
//===----------------------------------------------------------------------===//
#include "../ClangTidy.h"
#include "../ClangTidyModule.h"
#include "../ClangTidyModuleRegistry.h"
Add an argument comment checker to clang-tidy. This checks that parameters named in comments that appear before arguments in function and constructor calls match the parameter name used in the callee's declaration. For example: void f(int x, int y); void g() { f(/*y=*/0, /*z=*/0); } contains two violations of the policy, as the names 'x' and 'y' used in the declaration do not match names 'y' and 'z' used at the call site. I think there is significant value in being able to check/enforce this policy as a way of guarding against accidental API misuse and silent breakages caused by API changes. Although this pattern appears somewhat frequently in the LLVM codebase, this policy is not prescribed by the LLVM coding standards at the moment, so it lives under 'misc'. Differential Revision: http://llvm-reviews.chandlerc.com/D2914 llvm-svn: 204113
2014-03-18 12:46:45 +08:00
#include "ArgumentCommentCheck.h"
[clang-tidy] Add a checker for implicit bool conversion of a bool*. The goal is to find code like the example below, which is likely a typo where someone meant to write "if (*b)". bool *b = SomeFunction(); if (b) { // b never dereferenced } This checker naturally has a relatively high false positive rate so it applies some heuristics to avoid cases where the pointer is checked for nullptr before being written. Differential Revision: http://reviews.llvm.org/D4458 llvm-svn: 212797
2014-07-11 16:08:47 +08:00
#include "BoolPointerImplicitConversion.h"
[clang-tidy] Checker for inefficient use of algorithms on associative containers Summary: Associative containers implements some of the algorithms as methods which should be preferred to the algorithms in the algorithm header. The methods can take advantage of the order of the elements. Reviewers: alexfh Reviewed By: alexfh Subscribers: cfe-commits Differential Revision: http://reviews.llvm.org/D7246 llvm-svn: 228505
2015-02-08 03:54:19 +08:00
#include "InefficientAlgorithmCheck.h"
[clang-tidy] Add a checker for swapped arguments. This looks for swapped arguments by looking at implicit conversions of arguments void Foo(int, double); Foo(1.0, 3); // Most likely a bug llvm-svn: 212942
2014-07-14 22:24:30 +08:00
#include "SwappedArgumentsCheck.h"
[clang-tidy] Add a checker for code that looks like a delegate constructors but doesn't delegate. Summary: class Foo { Foo() { Foo(42); // oops } Foo(int); }; This is valid code but it does nothing and we can't emit a warning in clang because there might be side effects. The checker emits a warning for this pattern and also for base class initializers written in this style. There is some overlap with the unused-rtti checker but they follow different goals and fire in different places most of the time. Reviewers: alexfh, djasper Subscribers: cfe-commits Differential Revision: http://reviews.llvm.org/D4667 llvm-svn: 214397
2014-07-31 17:58:52 +08:00
#include "UndelegatedConstructor.h"
[clang-tidy] Add clang-tidy check for unique_ptr's reset+release -> move Replace x.reset(y.release()); with x = std::move(y); If y is rvalue, replace with x = y; instead. http://reviews.llvm.org/D6485 Patch by Alexey Sokolov! llvm-svn: 223460
2014-12-05 19:59:05 +08:00
#include "UniqueptrResetRelease.h"
[cleanup] Re-sort the #include lines with llvm/utils/sort_includes.py No functionality changed, this is just a mechanical cleanup to keep the order of #include lines consistent across the project. llvm-svn: 225976
2015-01-14 19:24:38 +08:00
#include "UnusedRAII.h"
Initial version of clang-tidy check to use override instead of virual. Review: http://reviews.llvm.org/D3688 llvm-svn: 208954
2014-05-16 17:30:09 +08:00
#include "UseOverride.h"
Added a module for checks not related to LLVM or Google coding style. llvm-svn: 202970
2014-03-05 21:14:32 +08:00
namespace clang {
namespace tidy {
class MiscModule : public ClangTidyModule {
public:
void addCheckFactories(ClangTidyCheckFactories &CheckFactories) override {
Unique-ptrify ClangTidyCheckFactories. Add a more convenient alternative to addCheckFactory: registerCheck. Reviewers: djasper Reviewed By: djasper Subscribers: cfe-commits Differential Revision: http://reviews.llvm.org/D5288 llvm-svn: 217489
2014-09-10 19:06:43 +08:00
CheckFactories.registerCheck<ArgumentCommentCheck>("misc-argument-comment");
CheckFactories.registerCheck<BoolPointerImplicitConversion>(
"misc-bool-pointer-implicit-conversion");
[clang-tidy] Checker for inefficient use of algorithms on associative containers Summary: Associative containers implements some of the algorithms as methods which should be preferred to the algorithms in the algorithm header. The methods can take advantage of the order of the elements. Reviewers: alexfh Reviewed By: alexfh Subscribers: cfe-commits Differential Revision: http://reviews.llvm.org/D7246 llvm-svn: 228505
2015-02-08 03:54:19 +08:00
CheckFactories.registerCheck<InefficientAlgorithmCheck>(
"misc-inefficient-algorithm");
Unique-ptrify ClangTidyCheckFactories. Add a more convenient alternative to addCheckFactory: registerCheck. Reviewers: djasper Reviewed By: djasper Subscribers: cfe-commits Differential Revision: http://reviews.llvm.org/D5288 llvm-svn: 217489
2014-09-10 19:06:43 +08:00
CheckFactories.registerCheck<SwappedArgumentsCheck>(
"misc-swapped-arguments");
CheckFactories.registerCheck<UndelegatedConstructorCheck>(
"misc-undelegated-constructor");
[clang-tidy] Add clang-tidy check for unique_ptr's reset+release -> move Replace x.reset(y.release()); with x = std::move(y); If y is rvalue, replace with x = y; instead. http://reviews.llvm.org/D6485 Patch by Alexey Sokolov! llvm-svn: 223460
2014-12-05 19:59:05 +08:00
CheckFactories.registerCheck<UniqueptrResetRelease>(
"misc-uniqueptr-reset-release");
Unique-ptrify ClangTidyCheckFactories. Add a more convenient alternative to addCheckFactory: registerCheck. Reviewers: djasper Reviewed By: djasper Subscribers: cfe-commits Differential Revision: http://reviews.llvm.org/D5288 llvm-svn: 217489
2014-09-10 19:06:43 +08:00
CheckFactories.registerCheck<UnusedRAIICheck>("misc-unused-raii");
CheckFactories.registerCheck<UseOverride>("misc-use-override");
Added a module for checks not related to LLVM or Google coding style. llvm-svn: 202970
2014-03-05 21:14:32 +08:00
}
};
// Register the MiscTidyModule using this statically initialized variable.
static ClangTidyModuleRegistry::Add<MiscModule>
X("misc-module", "Adds miscellaneous lint checks.");
// This anchor is used to force the linker to link in the generated object file
// and thus register the MiscModule.
volatile int MiscModuleAnchorSource = 0;
} // namespace tidy
} // namespace clang