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

88 lines
3.3 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] Assert related checkers This patch contains two assert related checkers. These checkers are the part of those that is being open sourced by Ericsson (http://lists.cs.uiuc.edu/pipermail/cfe-dev/2014-December/040520.html). The checkers: AssertSideEffect: /// \brief Finds \c assert() with side effect. /// /// The conition of \c assert() is evaluated only in debug builds so a condition /// with side effect can cause different behaviour in debug / relesase builds. StaticAssert: /// \brief Replaces \c assert() with \c static_assert() if the condition is /// evaluatable at compile time. /// /// The condition of \c static_assert() is evaluated at compile time which is /// safer and more efficient. http://reviews.llvm.org/D7375 Patch by Szabolcs Sipos! llvm-svn: 230943
2015-03-02 18:46:43 +08:00
#include "AssertSideEffectCheck.h"
Verify assign operator signatures. Summary: Warn when the return type of assign operators is not Class&. Reviewers: alexfh Subscribers: cfe-commits Differential Revision: http://reviews.llvm.org/D6667 llvm-svn: 228583
2015-02-10 01:50:40 +08:00
#include "AssignOperatorSignatureCheck.h"
[clang-tidy] Refactor: Rename clang-tidy misc check files and classes to follow naming conventions Classes are named WhateverCheck, files are named WhateverCheck.cpp and WhateverCheck.h. http://reviews.llvm.org/D8145 Patch by Richard Thomson! llvm-svn: 231648
2015-03-09 19:48:54 +08:00
#include "BoolPointerImplicitConversionCheck.h"
[clang-tidy] Checker for inaccurate use of erase() method. Algorithms like remove() does not actually remove any element from the container but returns an iterator to the first redundant element at the end of the container. These redundant elements must be removed using the erase() method. This check warns when not all of the elements will be removed due to using an inappropriate overload. Reviewer: alexfh Differential Revision: http://reviews.llvm.org/D7496 llvm-svn: 228679
2015-02-10 17:14:26 +08:00
#include "InaccurateEraseCheck.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 checker that warns about missing parentheses in macros * calculations in the replacement list should be inside parentheses * macro arguments should be inside parentheses llvm-svn: 239820
2015-06-16 22:27:31 +08:00
#include "MacroParenthesesCheck.h"
clang-tidy: Add checker that warn when macro argument with side effects is repeated in the macro llvm-svn: 239909
2015-06-17 22:19:35 +08:00
#include "MacroRepeatedSideEffectsCheck.h"
Add a new clang-tidy check (misc-move-constructor-init) that diagnoses move constructor initializations that call copy constructors instead of move constructors. llvm-svn: 245571
2015-08-20 23:52:52 +08:00
#include "MoveConstructorInitCheck.h"
[clang-tidy] Renamed misc-noexcept-move-ctors to misc-noexcept-move-constructor llvm-svn: 238326
2015-05-27 22:24:11 +08:00
#include "NoexceptMoveConstructorCheck.h"
[clang-tidy] Add misc-sizeof-container check to find sizeof() uses on stl containers. Summary: sizeof(some_std_string) is likely to be an error. This check finds this pattern and suggests using .size() instead. Reviewers: djasper, klimek, aaron.ballman Subscribers: aaron.ballman, cfe-commits Differential Revision: http://reviews.llvm.org/D12759 llvm-svn: 247297
2015-09-11 00:37:46 +08:00
#include "SizeofContainerCheck.h"
[clang-tidy] Assert related checkers This patch contains two assert related checkers. These checkers are the part of those that is being open sourced by Ericsson (http://lists.cs.uiuc.edu/pipermail/cfe-dev/2014-December/040520.html). The checkers: AssertSideEffect: /// \brief Finds \c assert() with side effect. /// /// The conition of \c assert() is evaluated only in debug builds so a condition /// with side effect can cause different behaviour in debug / relesase builds. StaticAssert: /// \brief Replaces \c assert() with \c static_assert() if the condition is /// evaluatable at compile time. /// /// The condition of \c static_assert() is evaluated at compile time which is /// safer and more efficient. http://reviews.llvm.org/D7375 Patch by Szabolcs Sipos! llvm-svn: 230943
2015-03-02 18:46:43 +08:00
#include "StaticAssertCheck.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] Refactor: Rename clang-tidy misc check files and classes to follow naming conventions Classes are named WhateverCheck, files are named WhateverCheck.cpp and WhateverCheck.h. http://reviews.llvm.org/D8145 Patch by Richard Thomson! llvm-svn: 231648
2015-03-09 19:48:54 +08:00
#include "UniqueptrResetReleaseCheck.h"
Add misc-unused-alias-decls check that currently finds unused namespace alias declarations. In the future, we might want to reuse it to also fine unsed using declarations and such. llvm-svn: 243754
2015-08-01 00:08:10 +08:00
#include "UnusedAliasDeclsCheck.h"
Initial version of clang-tidy check to find and fix unused parameters. Also see: llvm.org/PR24180. llvm-svn: 242654
2015-07-20 09:06:44 +08:00
#include "UnusedParametersCheck.h"
[clang-tidy] Refactor: Rename clang-tidy misc check files and classes to follow naming conventions Classes are named WhateverCheck, files are named WhateverCheck.cpp and WhateverCheck.h. http://reviews.llvm.org/D8145 Patch by Richard Thomson! llvm-svn: 231648
2015-03-09 19:48:54 +08:00
#include "UnusedRAIICheck.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 {
[clang-tidy] Refactor: Move misc clang-tidy checks to namespace clang::tidy::misc clang-tidy checks are organized into modules. This refactoring moves the misc module checks into the namespace clang::tidy::misc http://reviews.llvm.org/D7996 Patch by Richard Thomson! llvm-svn: 230950
2015-03-02 20:25:03 +08:00
namespace misc {
Added a module for checks not related to LLVM or Google coding style. llvm-svn: 202970
2014-03-05 21:14:32 +08:00
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");
[clang-tidy] Assert related checkers This patch contains two assert related checkers. These checkers are the part of those that is being open sourced by Ericsson (http://lists.cs.uiuc.edu/pipermail/cfe-dev/2014-December/040520.html). The checkers: AssertSideEffect: /// \brief Finds \c assert() with side effect. /// /// The conition of \c assert() is evaluated only in debug builds so a condition /// with side effect can cause different behaviour in debug / relesase builds. StaticAssert: /// \brief Replaces \c assert() with \c static_assert() if the condition is /// evaluatable at compile time. /// /// The condition of \c static_assert() is evaluated at compile time which is /// safer and more efficient. http://reviews.llvm.org/D7375 Patch by Szabolcs Sipos! llvm-svn: 230943
2015-03-02 18:46:43 +08:00
CheckFactories.registerCheck<AssertSideEffectCheck>(
"misc-assert-side-effect");
Verify assign operator signatures. Summary: Warn when the return type of assign operators is not Class&. Reviewers: alexfh Subscribers: cfe-commits Differential Revision: http://reviews.llvm.org/D6667 llvm-svn: 228583
2015-02-10 01:50:40 +08:00
CheckFactories.registerCheck<AssignOperatorSignatureCheck>(
"misc-assign-operator-signature");
[clang-tidy] Refactor: Rename clang-tidy misc check files and classes to follow naming conventions Classes are named WhateverCheck, files are named WhateverCheck.cpp and WhateverCheck.h. http://reviews.llvm.org/D8145 Patch by Richard Thomson! llvm-svn: 231648
2015-03-09 19:48:54 +08:00
CheckFactories.registerCheck<BoolPointerImplicitConversionCheck>(
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
"misc-bool-pointer-implicit-conversion");
[clang-tidy] Checker for inaccurate use of erase() method. Algorithms like remove() does not actually remove any element from the container but returns an iterator to the first redundant element at the end of the container. These redundant elements must be removed using the erase() method. This check warns when not all of the elements will be removed due to using an inappropriate overload. Reviewer: alexfh Differential Revision: http://reviews.llvm.org/D7496 llvm-svn: 228679
2015-02-10 17:14:26 +08:00
CheckFactories.registerCheck<InaccurateEraseCheck>(
"misc-inaccurate-erase");
[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");
clang-tidy: Add checker that warns about missing parentheses in macros * calculations in the replacement list should be inside parentheses * macro arguments should be inside parentheses llvm-svn: 239820
2015-06-16 22:27:31 +08:00
CheckFactories.registerCheck<MacroParenthesesCheck>(
"misc-macro-parentheses");
clang-tidy: Add checker that warn when macro argument with side effects is repeated in the macro llvm-svn: 239909
2015-06-17 22:19:35 +08:00
CheckFactories.registerCheck<MacroRepeatedSideEffectsCheck>(
"misc-macro-repeated-side-effects");
Add a new clang-tidy check (misc-move-constructor-init) that diagnoses move constructor initializations that call copy constructors instead of move constructors. llvm-svn: 245571
2015-08-20 23:52:52 +08:00
CheckFactories.registerCheck<MoveConstructorInitCheck>(
"misc-move-constructor-init");
[clang-tidy] Renamed misc-noexcept-move-ctors to misc-noexcept-move-constructor llvm-svn: 238326
2015-05-27 22:24:11 +08:00
CheckFactories.registerCheck<NoexceptMoveConstructorCheck>(
"misc-noexcept-move-constructor");
[clang-tidy] Add misc-sizeof-container check to find sizeof() uses on stl containers. Summary: sizeof(some_std_string) is likely to be an error. This check finds this pattern and suggests using .size() instead. Reviewers: djasper, klimek, aaron.ballman Subscribers: aaron.ballman, cfe-commits Differential Revision: http://reviews.llvm.org/D12759 llvm-svn: 247297
2015-09-11 00:37:46 +08:00
CheckFactories.registerCheck<SizeofContainerCheck>(
"misc-sizeof-container");
[clang-tidy] Assert related checkers This patch contains two assert related checkers. These checkers are the part of those that is being open sourced by Ericsson (http://lists.cs.uiuc.edu/pipermail/cfe-dev/2014-December/040520.html). The checkers: AssertSideEffect: /// \brief Finds \c assert() with side effect. /// /// The conition of \c assert() is evaluated only in debug builds so a condition /// with side effect can cause different behaviour in debug / relesase builds. StaticAssert: /// \brief Replaces \c assert() with \c static_assert() if the condition is /// evaluatable at compile time. /// /// The condition of \c static_assert() is evaluated at compile time which is /// safer and more efficient. http://reviews.llvm.org/D7375 Patch by Szabolcs Sipos! llvm-svn: 230943
2015-03-02 18:46:43 +08:00
CheckFactories.registerCheck<StaticAssertCheck>(
"misc-static-assert");
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] Refactor: Rename clang-tidy misc check files and classes to follow naming conventions Classes are named WhateverCheck, files are named WhateverCheck.cpp and WhateverCheck.h. http://reviews.llvm.org/D8145 Patch by Richard Thomson! llvm-svn: 231648
2015-03-09 19:48:54 +08:00
CheckFactories.registerCheck<UniqueptrResetReleaseCheck>(
[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
"misc-uniqueptr-reset-release");
Add misc-unused-alias-decls check that currently finds unused namespace alias declarations. In the future, we might want to reuse it to also fine unsed using declarations and such. llvm-svn: 243754
2015-08-01 00:08:10 +08:00
CheckFactories.registerCheck<UnusedAliasDeclsCheck>(
"misc-unused-alias-decls");
Initial version of clang-tidy check to find and fix unused parameters. Also see: llvm.org/PR24180. llvm-svn: 242654
2015-07-20 09:06:44 +08:00
CheckFactories.registerCheck<UnusedParametersCheck>(
"misc-unused-parameters");
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");
Added a module for checks not related to LLVM or Google coding style. llvm-svn: 202970
2014-03-05 21:14:32 +08:00
}
};
[clang-tidy] Refactor: Move misc clang-tidy checks to namespace clang::tidy::misc clang-tidy checks are organized into modules. This refactoring moves the misc module checks into the namespace clang::tidy::misc http://reviews.llvm.org/D7996 Patch by Richard Thomson! llvm-svn: 230950
2015-03-02 20:25:03 +08:00
} // namespace misc
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.
[clang-tidy] Refactor: Move misc clang-tidy checks to namespace clang::tidy::misc clang-tidy checks are organized into modules. This refactoring moves the misc module checks into the namespace clang::tidy::misc http://reviews.llvm.org/D7996 Patch by Richard Thomson! llvm-svn: 230950
2015-03-02 20:25:03 +08:00
static ClangTidyModuleRegistry::Add<misc::MiscModule>
Added a module for checks not related to LLVM or Google coding style. llvm-svn: 202970
2014-03-05 21:14:32 +08:00
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