7 Commits

Author	SHA1	Message	Date
Malcolm Parsons	91e2c2addc	[clang-tidy] Ignore ExprWithCleanups when looking for else-after-throw ... Summary: The readability-else-after-return check was not warning about an else after a throw of an exception that had arguments that needed to be cleaned up. Reviewers: aaron.ballman, alexfh, djasper Reviewed By: aaron.ballman Subscribers: lebedev.ri, klimek, xazax.hun, cfe-commits Differential Revision: https://reviews.llvm.org/D40505 llvm-svn: 319174	2017-11-28 14:57:47 +00:00
Malcolm Parsons	c35be36962	[clang-tidy] Fixed readability-else-after-return for cascade statements ... Summary: Fix generated by this check changed program semantics in the case where 'if' was a part (direct child) of other statement. Fixes PR30652. Patch by Paweł Żukowski. Reviewers: malcolm.parsons, alexfh, djasper Subscribers: mgehre, omtcyfz, cfe-commits Differential Revision: https://reviews.llvm.org/D26125 llvm-svn: 285999	2016-11-04 16:32:14 +00:00
Kirill Bobyrev	34789edbbf	[clang-tidy] enhance readability-else-after-return ... `readability-else-after-return` only warns about `return` calls, but LLVM Coding Standars stat that `throw`, `continue`, `goto`, etc after `return` calls are bad, too. Reviwers: alexfh, aaron.ballman Differential Revision: https://reviews.llvm.org/D23265 llvm-svn: 278257	2016-08-10 18:05:47 +00:00
Etienne Bergeron	8311c446fc	[clang-tidy] Refactoring of FixHintUtils ... Summary: Refactor some checkers to use the tooling re-writing API. see: http://reviews.llvm.org/D19941 Reviewers: klimek, alexfh Subscribers: cfe-commits Differential Revision: http://reviews.llvm.org/D19807 llvm-svn: 269210	2016-05-11 17:38:22 +00:00
Daniel Jasper	a73aed0c9b	clang-tidy: [readability-else-after-return] Fix false positive. This ... might be a little too strict now, but better be too strict than do the wrong thing. llvm-svn: 235932	2015-04-27 22:42:20 +00:00
Alexander Kornienko	8384d491d8	[clang-tidy] Refactor: Move readability checks to namespace clang::tidy::readability ... clang-tidy checks are organized into modules. This refactoring moves the readability module checks into the namespace clang::tidy::readability http://reviews.llvm.org/D7997 Patch by Richard Thomson! llvm-svn: 230946	2015-03-02 11:43:00 +00:00
Daniel Jasper	3b6018b9f6	clang-tidy: Add initial check for "Don't use else after return". ... As per the LLVM coding standards: http://llvm.org/docs/CodingStandards.html#don-t-use-else-after-a-return Initial version, probably still quite a bit to do until this is really useful. Review: http://reviews.llvm.org/D6927 llvm-svn: 226025	2015-01-14 19:37:54 +00:00