summaryrefslogtreecommitdiffstats
path: root/clang-tools-extra/unittests/clang-tidy/ReadabilityModuleTest.cpp
Commit message (Collapse)AuthorAgeFilesLines
* [clang-tidy] Add a test.Alexander Kornienko2017-08-101-3/+9
| | | | llvm-svn: 310586
* [clang-tidy] Fix readability-braces-around-statements false positiveMarek Kurdej2016-10-141-16/+30
| | | | | | | | | | | | | | | | | Summary: This fixes a false-positive e.g. when string literals are returned from if statement. This patch includes as well a small fix to includes and renames of the test suite that collided with the name of the check. Reviewers: alexfh, hokein Subscribers: hokein Tags: #clang-tools-extra Differential Revision: https://reviews.llvm.org/D25558 llvm-svn: 284212
* Create interfaces and tests for the overlapping replacements fix in clang-tidy.Angel Garcia Gomez2015-10-061-1/+1
| | | | | | | | | | | | Summary: No changes in clang-tidy yet. Reviewers: klimek Subscribers: alexfh, cfe-commits Differential Revision: http://reviews.llvm.org/D13469 llvm-svn: 249402
* [clang-tidy] Fix PR22785.Alexander Kornienko2015-09-091-0/+21
| | | | | | | | | | | Fix http://llvm.org/PR22785. Bug 22785 - readability-braces-around-statements doesn't work well with macros. http://reviews.llvm.org/D12729 Patch by Marek Kurdej! llvm-svn: 247163
* [clang-tidy] Force braces around leaf 'else if' for consistency.Samuel Benzaquen2015-06-041-0/+13
| | | | | | | | | | | | | | Summary: Force braces around leaf 'else if' for consistency. This complements r233697. Reviewers: alexfh Subscribers: curdeius, cfe-commits Differential Revision: http://reviews.llvm.org/D10245 llvm-svn: 239054
* Force braces on the else branch if they are being added to the if branch.Samuel Benzaquen2015-03-311-0/+21
| | | | | | | | | | | | | | Summary: Force braces on the else branch if they are being added to the if branch. This ensures consistency in the transformed code. Reviewers: alexfh Subscribers: cfe-commits Differential Revision: http://reviews.llvm.org/D8708 llvm-svn: 233697
* [clang-tidy] Slighly clarified a comment.Alexander Kornienko2015-03-051-1/+2
| | | | llvm-svn: 231370
* [clang-tidy] Replace unrecognized namespace ending comments.Alexander Kornienko2015-03-051-5/+5
| | | | | | | | | | | | | | | | | | | | | | Summary: Replace unrecognized namespace ending comments. This will help in particular when a namespace ending comment is mistyped or doesn't fit the regexp for other reason, e.g.: namespace a { namespace b { namespace { } // anoynmous namespace } // b } // namesapce a Reviewers: djasper Reviewed By: djasper Subscribers: curdeius, cfe-commits Differential Revision: http://reviews.llvm.org/D8078 llvm-svn: 231369
* [clang-tidy] Organized clang-tidy unit tests. NFC.Alexander Kornienko2015-03-021-0/+330
| | | | | | | | | * Moved unit tests for BracesAroundStatementsCheck to ReadabilityModuleTest.cpp. * Moved EXPECT_NO_CHANGES macro to ClangTidyTest.h to avoid defining it three times. llvm-svn: 230947
* [clang-tidy] Bring order to check registration.Alexander Kornienko2014-10-261-0/+99
Summary: Register readability checks in a separate module. Renamed the checks and test file names accordingly. Reviewers: djasper, klimek Reviewed By: klimek Subscribers: curdeius, cfe-commits Differential Revision: http://reviews.llvm.org/D5936 llvm-svn: 220631
OpenPOWER on IntegriCloud